Skip to content

[PropertyInfo] ensure docblock compatibility with PhpStan's docblock parser #48215

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
Nov 15, 2022

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Nov 15, 2022

Q A
Branch? 4.4
Bug fix? no
New feature? no
Deprecations? no
Tickets
License MIT
Doc PR

In their next releases both packages phpdocumentor/reflection-docblock and phpdocumentor/type-resolver will start using the docblock parser from PhpStan.

In their next releases both packages phpdocumentor/reflection-docblock and
phpdocumentor/type-resolver will start using the docblock parser from PhpStan.
@xabbuh
Copy link
Member Author

xabbuh commented Nov 15, 2022

The alternative solution would be to conflict with the upcoming minor releases of both package or see if phpDocumentor/ReflectionDocBlock#343 (comment) can be fixed over there instead.

@stof
Copy link
Member

stof commented Nov 15, 2022

Fixing that makes sense to me as it is only a test fixture anyway.

However, I think phpDocumentor should still fix this before release their new version (or make it a new major version if they break some supported types)

@xabbuh xabbuh merged commit 0eebba6 into symfony:4.4 Nov 15, 2022
@xabbuh xabbuh deleted the property-info-docblock-format branch November 15, 2022 13:11
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.

3 participants