-
Notifications
You must be signed in to change notification settings - Fork 137
Update dependency nikic/php-parser to v5 - autoclosed #1387
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
Conversation
433cf0e to
e40037e
Compare
e40037e to
b334009
Compare
0b01c6b to
b3d0f80
Compare
|
If we want to move this forward from our end, it's probably feasible via extracting test dependencies to their own directory. That becomes a pain with the test setup though, so unsure about the chicken-egg situation, for now. Will sleep over it a few times 🤔 |
c90fc01 to
91b72da
Compare
3b7e868 to
7f43414
Compare
|
It's not a big deal, but it'd be nice if you could support version 5. composer why nikic/php-parser
phpunit/php-code-coverage 9.2.30 requires nikic/php-parser (^4.18 || ^5.0)
roave/better-reflection 6.23.0 requires nikic/php-parser (^4.18.0)
sebastian/complexity 2.0.3 requires nikic/php-parser (^4.18 || ^5.0)
sebastian/lines-of-code 1.0.4 requires nikic/php-parser (^4.18 || ^5.0)
symfony/maker-bundle v1.53.0 requires nikic/php-parser (^4.18|^5.0) |
4783453 to
80c5b51
Compare
|
While I don't understand rennovate, it's working in 'dev' for me now, thanks! |
|
In theory, this is fixable via Roave/infection-static-analysis-plugin#489 (and some additional testing), but I'm stuck over there due to tests failing for a missing coverage driver. What is confusing me is why the mainline branch and the PR have different build environments (within the same day): Roave/infection-static-analysis-plugin#489 (comment) |
80c5b51 to
f8ab759
Compare
41b0f37 to
eee58f4
Compare
90c6f4f to
5f015f3
Compare
a7eca26 to
5fa25fb
Compare
5fa25fb to
d2c632b
Compare
8d02034 to
6bed0e7
Compare
6bed0e7 to
5603f73
Compare
5603f73 to
881a1f5
Compare
| datasource | package | from | to | | ---------- | ---------------- | ------ | ----- | | packagist | nikic/php-parser | 4.18.0 | 5.0.2 |
881a1f5 to
29ab04c
Compare
This PR contains the following updates:
^4.18.0->^5.0.2Release Notes
nikic/PHP-Parser (nikic/php-parser)
v5.0.2Compare Source
Fixed
printer.
Changed
Parserobjects. This means that no longer used parser objects areimmediately destroyed now, instead of requiring cycle GC.
PhpVersion::getNewestSupported()to report PHP 8.3 instead of PHP 8.2.v5.0.1Compare Source
Changed
T_*compatibility tokenswith incorrect type (such as string instead of int). This would lead to
TypeErrors down theline. Now an
Errorwill be thrown early to indicate the problem.v5.0.0Compare Source
See UPGRADE-5.0 for detailed migration instructions.
Fixed
PropertyItemandUseItem.Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
Read more about the use of Renovate Bot within
ocramius/*projects.