Skip to content

CI: pin PrestaShop 9.1.5 and add 9.2.x to the PHPStan matrix - #183

Merged
jolelievre merged 2 commits into
PrestaShop:devfrom
mattgoud:ci/pin-9.1.5-add-9.2.x
Aug 28, 2026
Merged

CI: pin PrestaShop 9.1.5 and add 9.2.x to the PHPStan matrix#183
jolelievre merged 2 commits into
PrestaShop:devfrom
mattgoud:ci/pin-9.1.5-add-9.2.x

Conversation

@mattgoud

Copy link
Copy Markdown
Contributor
Questions Answers
Description? Two CI changes. First, the PHPStan matrix is pinned on the 9.1.5 tag instead of 9.1.4: 9.1.5 is the last release of the 9.1 line, since the 9.1.x branch was deleted from the core once it shipped. Second, PrestaShop 9.2.x joins the matrix, so this module has the same coverage as pagesnotfound and blockreassurance, which already had it.
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? Part of PrestaShop/PrestaShop#41648
How to test? CI only. The two 9.1.5 and the two new 9.2.x PHPStan jobs must be green.
Sponsor company @PrestaShopCorp

The 9.1.4 pin was merged before 9.1.5 was released (2026-08-18), hence the follow-up.

tests/php/phpstan/phpstan-9.2.x.neon is a copy of the 9.1.5 one, so any suppression the module legitimately keeps applies to 9.2.x as well.

9.1.5 is the last release of the 9.1 line: the 9.1.x branch was deleted
from the core once it shipped, so nothing newer will come out of it.
Same coverage as pagesnotfound and blockreassurance, which already had it.
@ps-jarvis ps-jarvis added the Waiting for QA Status: Action required, Waiting for test feedback label Aug 28, 2026
@jolelievre
jolelievre merged commit 96c8c46 into PrestaShop:dev Aug 28, 2026
14 checks passed
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Aug 28, 2026
@github-project-automation github-project-automation Bot moved this from To be tested to Merged in PR Dashboard Aug 28, 2026
@ps-jarvis ps-jarvis moved this from Merged to To be tested in PR Dashboard Aug 28, 2026
@ps-jarvis

Copy link
Copy Markdown

PR merged, well done!

Message to @PrestaShop/committers: do not forget to milestone it before the merge.

@mattgoud mattgoud removed the Waiting for QA Status: Action required, Waiting for test feedback label Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To be tested

Development

Successfully merging this pull request may close these issues.

3 participants