Add nikic/php-parser v5 support#11035
Conversation
5b4cba4 to
6358029
Compare
6b17670 to
e7c21a4
Compare
Thanks. I added the CI "lowest" with php parser 4 and fixed the build. |
e7c21a4 to
b6282ae
Compare
|
Friendly ping @weirdan if you have time to take a look (or can ping another maintainer ?). |
|
Yeah, it's on my todo list. The thing is big, so it'll take me some time. |
|
Since we expose parts of PhpParser in our API this cannot go into 5.x line. So instead of merging this I'm going to fork 6.x from the current 5.x (not master) and then use this PR as a base for PhpParser 5.x upgrade. 6.x will be a transitional, hopefully short-lived, series with the same platform requirements as 5.x. Thanks for all your work, this will make it easier. |
|
If the For example: "vimeo/psalm": "6.x-dev#0f796f1"Replace |
|
Any update on this? I'm unable to install |
|
In addition to the dependency conflicts with various packages, PHP 8.4 releases in about a week. Currently Psalm is unable to parse these constructs, let alone type check them, so Psalm users will be blocked from adopting these new language features. |
It's this + that it's blocking anything else using |
I installed phar with composer itself, but when I tried to install the laravel plugin, there was a conflict 🙃 |
|
+1, would love to see this one working |
|
+1 |
|
Please use directly dev-master until I tag a v6 (hopefully in January) |
|
JFYI: with property hooks (PHP 8.4) dev-master still not worked. |
|
Any ETA for this? Can't upgrade some dependencies because psalm depends on v4. |
|
Not merging this, please switch to dev-master/v6. |
|
For those following along, I've asked for an update on #11112. |
No description provided.