Skip to content

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

Merged
jolelievre merged 1 commit into
PrestaShop:devfrom
mattgoud:ci/pin-9.1.5
Aug 31, 2026
Merged

CI: pin PrestaShop 9.1.5 and add 9.2.x to the PHPStan matrix#30
jolelievre merged 1 commit into
PrestaShop:devfrom
mattgoud:ci/pin-9.1.5

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 gets the same coverage as the rest of the Epic.
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.

The neon rename is a git mv, since phpstan-config interpolates matrix.presta_version. 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.

9.2.x joins the matrix so this module has the same coverage as the others.
@ps-jarvis

Copy link
Copy Markdown

Hello @mattgoud!

This is your first pull request on ps_specials repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Aug 28, 2026
@mattgoud mattgoud self-assigned this Aug 28, 2026
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Aug 31, 2026
@jolelievre
jolelievre merged commit 78e9bb0 into PrestaShop:dev Aug 31, 2026
16 checks passed
@ps-jarvis

Copy link
Copy Markdown

PR merged, well done!

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

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