Skip to content

Pin PrestaShop 9.1.4 in CI instead of 9.1.x branch - #1278

Open
kpodemski wants to merge 1 commit into
PrestaShop:devfrom
kpodemski:ci/pin-prestashop-9.1.4
Open

Pin PrestaShop 9.1.4 in CI instead of 9.1.x branch#1278
kpodemski wants to merge 1 commit into
PrestaShop:devfrom
kpodemski:ci/pin-prestashop-9.1.4

Conversation

@kpodemski

Copy link
Copy Markdown
Contributor
Questions Answers
Description? Points the PHPStan CI matrix at the PrestaShop 9.1.4 tag instead of the 9.1.x branch, which is about to be closed. Also renames tests/php/phpstan/phpstan-9.1.x.neon to phpstan-9.1.4.neon, because the workflow derives the PHPStan config filename from the matrix value (phpstan-config: tests/php/phpstan/phpstan-${{ matrix.presta_version }}.neon) — without the rename the job would fail on a missing config. This matches the existing phpstan-9.0.3.neon, which is already pinned to a tag.
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? n/a
How to test? Check this PR's "PHP tests" workflow run: the two PHPStan (PHP 8.1 - 8.5) jobs that previously ran against 9.1.x must now show 9.1.4 in their matrix names and be green. No job should reference 9.1.x any more. The 9.0.3, develop and 8.2.x jobs are untouched.
Sponsor company n/a

Only CI configuration is affected — no module code changes.

The 9.1.x string still appears once, in a comment that describes which PHP versions the branch family supports; it has been reworded to # PrestaShop 9.1.x (pinned to the 9.1.4 tag) and develop support PHP 8.1 up to 8.5 so it stays accurate.

@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for review

Development

Successfully merging this pull request may close these issues.

2 participants