-
Notifications
You must be signed in to change notification settings - Fork 473
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
GithubAction: added job to verify tests fail without src/ changes #2029
Conversation
Some suggestions:
|
I think it works as expected regarding
missing atm is
(and of course the temp changes need to be reverted, as soon as we get everything running as expected) |
I think we are feature complete.
the job dependency graph if you are fine with the result, I will re-enable all lines I previously commented to speedup the feedback loop. |
If we agreed on this one, I can cleanup all the test code and remove the temporary changes |
e84aa7b
to
cce9f8e
Compare
squashed and rebased |
This pull request has been marked as ready for review. |
@ondrejmirtes is this PR useful? should I rebase it for 1.10.x? |
Refs laminas/laminas-continuous-integration-action#117
Refs phpstan/phpstan#8228 (comment)