-
-
Notifications
You must be signed in to change notification settings - Fork 155
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
Bump PHP-Parser #1354
Bump PHP-Parser #1354
Conversation
Did I get it right that we can't merge it? (because of that issue) |
No, we can't. Phar build is broken. |
I think I'll leave thing handing just on the off chance @theofidry can know what's up here. |
@maks-rafalko Now I can build it just fine locally. |
'T_NAME_RELATIVE', | ||
'T_MATCH', | ||
'T_NULLSAFE_OBJECT_OPERATOR', | ||
'T_ATTRIBUTE', |
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.
Taken from here:
PHP-Scooper does not know that it must whitelist these otherwise.
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.
Could this be added to https://github.com/humbug/php-scoper/blob/master/src/Reflector.php#L64?
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.
Absolutely. I thought there must be something like this, but couldn't find this straight away.
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.
👍
This should be not very hard to backport. |
* Whitelist PHP 8.0 constants in scoper.inc.php
* Whitelist PHP 8.0 constants in scoper.inc.php
* Bump PHP-Parser (#1354) * Whitelist PHP 8.0 constants in scoper.inc.php * Fix PHPStan warning
This PR:
PHP-Parser had an issue, but apparently it's gone in newer versions.It is still there: