-
-
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
Always lint test cases with the stricter process linter #5180
Conversation
c0e976b
to
f419c91
Compare
NB This also highlights a bunch of PHP 8 linting issues (which should be addressed in separate followup PRs). |
Thanks, looking good so far 👍 |
822197d
to
379376e
Compare
@SpacePossum I think this is ready for merge, followed by merges upwards, and then your other fixes PR for 2.16. After that, I will PR some GitHub actions stuff to the 2.15 branch. |
207dc35
to
1d2f24b
Compare
Thank you @GrahamCampbell great to see this in :) 👍 |
This will prevent invalid test cases like the ones removed by #5178.