-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Improve Symfony 5 compatibility #4685
Conversation
I can confirm this fixes the issue. Here is my reproducer: https://github.com/kbond/php-cs-fixer-2.16.1-reproducer |
Any chance of merging and tagging a new version with this soon? The PR is opened for almost a month already and it is blocking us from properly testing our package in CI (with different Symfony and PHP versions) -> trikoder/oauth2-bundle#141 |
Hello do you know when this fix will be released ? |
Thank you @keradus. |
This PR was merged into the 2.15 branch. Discussion ---------- Improve Symfony 5 compatibility Closes #4684 one can install contract on his own, even if he is using Sf version that does not rely on contracts, for that, we need to check if actually Sf component relies on contract, not only the fact that contract exists. TODO: - [x] look for other places when we check that contract exists and nothing more (edit by @julienfalque: only found one in the related test, not sure it's worth the hassle) Commits ------- 03f6fd5 Better Sf5 compatibility
Closes #4684
one can install contract on his own, even if he is using Sf version that does not rely on contracts,
for that, we need to check if actually Sf component relies on contract, not only the fact that contract exists.
TODO: