Skip to content

Align CI/CD with PrestaShop issue #41648#1238

Open
jf-viguier wants to merge 1 commit into
PrestaShop:devfrom
jf-viguier:ci/align-with-issue-41648
Open

Align CI/CD with PrestaShop issue #41648#1238
jf-viguier wants to merge 1 commit into
PrestaShop:devfrom
jf-viguier:ci/align-with-issue-41648

Conversation

@jf-viguier

@jf-viguier jf-viguier commented Jun 10, 2026

Copy link
Copy Markdown
Contributor
Questions Answers
Description? Aligns the module's CI/CD workflows with the conventions defined in PrestaShop/PrestaShop#41648, using blockreassurance as the reference module.
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket?
How to test? see below
Sponsor company Creabilis.com

Changes

PHP Linter (.github/workflows/php.yml)

  • Drop PHP 7.1 (no longer in scope)
  • Add PHP 8.5

PHPStan matrix (legacy job)

  • Remove PrestaShop 8.1.7 from matrix — only 8.2.x is kept, matching the convention

PHPUnit

  • Bump actions/checkout from v3.1.0 to v6 (consistency with other jobs)

PHPStan configs (tests/php/phpstan/)

  • Remove obsolete configs: phpstan-1.7.7.x.neon, phpstan-1.7.8.x.neon, phpstan-8.1.7.neon

Note

Dead-code cleanup for PrestaShop versions < 8.2 is not included here — it will be addressed in a separate PR as suggested by the issue.

Test plan

  • PHP linter job passes on PHP 7.2 → 8.5
  • PHPStan (PHP 7.4 - 8.1) passes on PS 8.2.x
  • PHPStan (PHP 8.1 - 8.5) passes on PS 9.1.x and develop
  • PHPUnit job still runs successfully

- Drop PHP 7.1 from linter matrix (no longer supported)
- Add PHP 8.5 to linter matrix
- Remove PrestaShop 8.1.7 from PHPStan matrix (only 8.2.x kept)
- Bump actions/checkout to v6 in PHPUnit job
- Remove obsolete PHPStan configs for PS 1.7.7.x, 1.7.8.x, 8.1.7

Reference: PrestaShop/PrestaShop#41648
@jf-viguier jf-viguier added this to the 4.0.5 milestone Jun 10, 2026

@Hlavtox Hlavtox left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AI not respecting template

@ps-jarvis ps-jarvis added the waiting for author Waiting for author's feedback label Jun 10, 2026
@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Jun 10, 2026
@jf-viguier

Copy link
Copy Markdown
Contributor Author

@Hlavtox PR template respected, thanks.

@jf-viguier jf-viguier removed the waiting for author Waiting for author's feedback label Jun 17, 2026
@jf-viguier jf-viguier requested a review from Hlavtox June 17, 2026 11:12
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.

4 participants