Upgrade ondram/ci-detector to 4.x #4617
Merged
+22
−16
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
infection is using ondram/ci-detector 4.x and has class-loading problems when the project-under-test uses a older version of this lib, see infection/infection#2671
upgrading to 4.x allows us to run infection from within build-infection:
before this PR, this step errored with:
so while the root problem should be addressed in the infection application, it does not hurt for us to upgrade ondram/ci-detector to 4.x - which requires PHP 7.4+. this is fine for PHPStan 2.x as we are also requiring 7.4+
breaking changes of 4.x seem to not affect us