-
Notifications
You must be signed in to change notification settings - Fork 106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to PHP7.4 #127
Update to PHP7.4 #127
Conversation
Anyone here who approved this PR tested it out yet? I’ve only lightly tested it but I have a fresh kid at home and less time for the next month or so :D |
Docker is just rebuilding, will comment the result shortly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working for me on my project. ./vessel php -v
shows PHP 7.4.1
👍
thanks!
…On Fri, Jan 3, 2020 at 16:16 James King ***@***.***> wrote:
***@***.**** approved this pull request.
Working for me on my project. ./vessel php -v shows PHP 7.4.1 👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#127?email_source=notifications&email_token=AADSDUYFP3LNRDTALF554GLQ362NJA5CNFSM4JTWDDKKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCQVAM5I#pullrequestreview-338298485>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADSDU6JQ5GKN3FYELEQM3DQ362NJANCNFSM4JTWDDKA>
.
|
Tested it from my side too! |
@fideloper: mate, this PR will ensure all dependencies are upto date: #131 |
@fideloper Can you do a new release so I can pull the latest code with php7.4? |
Yep, tagged |
Looks like xDebug is ready for release already too!