Skip to content

chore(php): run php-cs-fixer image on PHP 8.2 Alpine - #14

Closed
nicolasotmani wants to merge 1 commit into
masterfrom
chore/php-8.2-base
Closed

nicolasotmani wants to merge 1 commit into
masterfrom
chore/php-8.2-base

Conversation

@nicolasotmani

Copy link
Copy Markdown

What Does It Do?

Switch the CS Fixer GHCR image base from php:8.1.34-alpine to php:8.2-alpine, keep Fixer 3.13.0, set PHP_CS_FIXER_IGNORE_ENV=1 so 3.13.0 can run on PHP 8.2.

Context

Aligns the fixer runtime with the mobsuccess PHP 8.2 upgrade. Official Fixer PHP 8.2 support starts at ≥3.15.0 (ruleset bump — separate PR). Pin php:8.2.35-cli-alpine when Hub publishes it.

Test Plan

  • docker compose build locally
  • docker run … --version shows PHP 8.2.x + Fixer 3.13.0
  • Consumer make php-cs-fixer dry-run still works after GHCR publish from master

Made with Cursor

Keep Fixer 3.13.0 rules; set PHP_CS_FIXER_IGNORE_ENV until a Fixer >=3.15 bump.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI lite review requested due to automatic review settings October 21, 2026 15:12
@github-actions github-actions Bot added the chore Chore feature label Oct 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

No unresolved blocking issues were identified.

No open issues 🎉

What changed in this PR

Updates the PHP-CS-Fixer container to PHP 8.2 Alpine while retaining Fixer 3.13.0 compatibility.

Changes:

  • Switches the base image to php:8.2-alpine.
  • Sets PHP_CS_FIXER_IGNORE_ENV=1.
  • Documents the runtime and compatibility strategy.
File Description
README.md Documents the updated runtime configuration.
entrypoint.sh Enables Fixer execution on PHP 8.2.
Dockerfile Uses the PHP 8.2 Alpine base image.

Review effort: Lite

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Chore feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants