Skip to content
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 to PHP-Parser v4 #330

Merged
merged 3 commits into from
Apr 2, 2018
Merged

Update to PHP-Parser v4 #330

merged 3 commits into from
Apr 2, 2018

Conversation

carusogabriel
Copy link
Contributor

PHP-Parser v4 was released, and now we can remove our own fork of Roave/BetterReflection.

composer.json Outdated
"phpdocumentor/reflection-docblock": "^4.3",
"phpdocumentor/type-resolver": "^0.4",
"rector/better-reflection": "^3.0.6",
"roave/better-reflection": "dev-master",
Copy link
Contributor Author

@carusogabriel carusogabriel Feb 28, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TomasVotruba Thanks, didn't know that one

@TomasVotruba
Copy link
Member

Horray! Party time :) I'll get to this tomorrow

composer.json Outdated
"phpdocumentor/reflection-docblock": "^4.3",
"phpdocumentor/type-resolver": "^0.4",
"rector/better-reflection": "^3.0.6",
"roave/better-reflection": "dev-master#dd2c2d0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To really use stable dependencies, here use rector/better-reflection and update it first to nikic/php-parser 4.0.
I'll release new tag for it then

@TomasVotruba TomasVotruba changed the title [WIP] Update to PHP-Parser v4 Update to PHP-Parser v4 Apr 2, 2018
@TomasVotruba TomasVotruba merged commit 0c835da into master Apr 2, 2018
@TomasVotruba TomasVotruba deleted the php-parser-v4 branch April 2, 2018 09:10
@TomasVotruba
Copy link
Member

Thank you Gabriel

@carusogabriel
Copy link
Contributor Author

I'll track Roave/BetterReflection#410, so when that gets merge, we can remove our fork 👍

@TomasVotruba
Copy link
Member

TomasVotruba commented Apr 2, 2018

Seeing the stuck discussion - though I suggested working solution - it might take many months.
We don't need that really :)

All these issues are more relevant: https://github.com/rectorphp/rector/issues

TomasVotruba added a commit that referenced this pull request Jun 29, 2021
rectorphp/rector-src@2a394c7 Reflection cleanup + inline union types for couple of param docs (#330)
echo511 pushed a commit to echo511/rector that referenced this pull request Sep 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants