-
-
Notifications
You must be signed in to change notification settings - Fork 68
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
Drupal 11 compatibility #309
Conversation
Wow, thanks! If this is ready I can merge |
I believe it finally is! :) Is this script still useful on D9+ sites? (At a minimum, it fixes some of the CI issues.) |
I've been recommending folks use PHPStan directly, especially since adding |
If this won't need to be installed on D11, feel free to pull out the Symfony version constraint updates. We can look into just using PHPStan directly and remove it from our build tool-chain for the D11 update. |
Oh, I think others still use this and your PR is valuable. In my FLDC talk I explained how to use PHPStan directly: http://2024.fldrupal.camp/sites/default/files/2024-02/Tighten%20up%20your%20Drupal%20code%20using%20PHPStan%20-%20FLDC%202024/index.pdf |
D11 uses Symfony 7.0. Update the Symfony version constraint to also allow version 7.0 and higher.
Also