Repository navigation
(Changed) Build and analyse on PHP 8.2 to 8.5 - #11
Merged
MarjovanLier merged 2 commits intoOct 27, 2026
Merged
Conversation
The package requires PHP ^8.2, but CI built only on 8.2 and 8.3, so nothing showed that the library works on 8.4 and 8.5. The matrix now runs every build step on all four versions. With no lock committed, 8.4 and 8.5 resolve PHPUnit 13 and Infection 0.35. Claude-Session: 3d0f6fda-56b2-4222-80a6-58ded8ac9989 Signed-off-by: Marjo Wenzel van Lier <marjo.vanlier@gmail.com>
PHPStan was pinned to PHP 8.2, so it judged the code only by 8.2 behaviour, while the package also supports 8.3, 8.4 and 8.5. A min/max phpVersion range makes it check the code against every supported version. Claude-Session: 3d0f6fda-56b2-4222-80a6-58ded8ac9989 Signed-off-by: Marjo Wenzel van Lier <marjo.vanlier@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
The package requires PHP ^8.2, but CI built only on 8.2 and 8.3 and PHPStan judged the code only by 8.2 behaviour, so nothing showed that the library works on 8.4 and 8.5. The CI matrix now runs every build step on 8.2, 8.3, 8.4 and 8.5; because no lock is committed, 8.4 and 8.5 resolve PHPUnit 13 and Infection 0.35. PHPStan now takes a min/max
phpVersionrange from 8.2 to 8.5 instead of a single 8.2 pin, so it checks the code against every supported version. Before the push, every build step passed on all four versions in the official PHP CLI images with Xdebug, with 88 tests and a 95% covered-code mutation score on each. The minimum stays PHP 8.2 and the Composer constraint stays^8.2, so installs are unchanged while 8.4 and 8.5 are now tested.