Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
[PropertyInfo] Fix PhpStanExtractor added version#18399
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
carsonbot commentedJun 9, 2023
Hey! Oh no, it looks like you have made this PR towards a branch that is not maintained anymore. :/ Cheers! Carsonbot |
MrYamous commentedJun 10, 2023
Nice catch, i think it can be rebased on 5.4 :) |
colinodell commentedJun 10, 2023
This section of the documentation wasn't added until 6.1:#16955 (comment) I'm not sure why that decision was made, so I'm hesitant to backport this to 5.4. |
javiereguiluz commentedJun 12, 2023
Colin, thanks a lot for fixing this. Sadly, this was a bit tricky:
So, while merging, I just removed the entire Thanks! |
PhpStanExtractorwas actually added in 5.4:symfony/symfony#40457 (comment)