-
-
Notifications
You must be signed in to change notification settings - Fork 100
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 php-scoper + bump min PHP version to 7.3 #499
Conversation
Travis failure is related to nikic/PHP-Parser#701 |
FYI there is also some tests failing because of a breaking change in Composer 2.0.5 ( |
Is it something we want now or should I go ahead with another release first? |
cc41d5d
to
d9c1839
Compare
No, not yet. Go ahead with another release :-) |
d9c1839
to
e95500c
Compare
e95500c
to
d146a21
Compare
There is an issue with scoping php-parser with the current php-scoper version. I could fix it with humbug/php-scoper#437. But I had bump min PHP version to 7.3 as it has been done in php-scoper too. |
@theofidry Bump min PHP version to 7.3 causes tests failures. Can we revert php-scoper or have a branch with PHP 7.2 support? |
@smoench sorry I'm not sure to understand, why do we need 7.2 support? |
Composer won't let us install the latest php-scoper version as in box's composer.json the platform config for PHP is fixed to 7.2.9. |
But if we drop 7.2 (which was the plan) then this value should be bumped right? |
Docker e2e tests had to be updated. Works now. |
Waiting for humbug/php-scoper#437 to be merged and a new php-scoper release :) |
44fbb45
to
3b751de
Compare
3b751de
to
399ca62
Compare
Thank you @theofidry. I've updated the composer.json :-) |
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.
LGTM overall, thanks!
This will fix also the issue with scoping php-parser in #489