Skip to content

[Translation] drop support for nikic/php-parser 4 #60255

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

Merged
merged 1 commit into from
Apr 23, 2025

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Apr 23, 2025

Q A
Branch? 7.3
Bug fix? no
New feature? no
Deprecations? no
Issues
License MIT

Extracted from #58370, background: PHPUnit 11 is no longer compatible with nikic/php-parser 4 (see #58370 (comment)).

@xabbuh xabbuh requested a review from welcoMattic as a code owner April 23, 2025 05:20
@carsonbot carsonbot added this to the 7.3 milestone Apr 23, 2025
@carsonbot carsonbot changed the title [Translation] drop support for nikic/php-parser 4 [Translation] drop support for nikic/php-parser 4 Apr 23, 2025
@fabpot
Copy link
Member

fabpot commented Apr 23, 2025

Thank you @xabbuh.

@fabpot fabpot merged commit 2ae7fe3 into symfony:7.3 Apr 23, 2025
5 of 11 checks passed
@xabbuh xabbuh deleted the nikic-php-parser-5 branch April 23, 2025 06:12
@stof
Copy link
Member

stof commented Apr 24, 2025

A conflict rule probably needs to be added (the require-dev bump won't affect projects)

@stof
Copy link
Member

stof commented Apr 24, 2025

btw, this does not clean the code related to the php-parser v4 (it only makes it untested)

@xabbuh
Copy link
Member Author

xabbuh commented Apr 24, 2025

btw, this does not clean the code related to the php-parser v4 (it only makes it untested)

If I am not mistaken, we don't have v4-specific code since #53475.

@stof
Copy link
Member

stof commented Apr 24, 2025

ah indeed. I forgot that PR.

chalasr added a commit that referenced this pull request Apr 24, 2025
This PR was merged into the 7.3 branch.

Discussion
----------

[Translation] conflict with nikic/php-parser 4

| Q             | A
| ------------- | ---
| Branch?       | 7.3
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | see #60255 (comment)
| License       | MIT

Commits
-------

9cb5585 conflict with nikic/php-parser 4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants