Skip to content

ci: test against PHP 8.6 - #25

Open
donatj wants to merge 3 commits into
masterfrom
ci/add-php-8-6
Open

donatj wants to merge 3 commits into
masterfrom
ci/add-php-8-6

Conversation

@donatj

@donatj donatj commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

PHP 8.6 is in beta, and PHP-CS-Fixer currently rejects the beta runtime before CI can complete its checks.

Adds PHP 8.6 to the CI matrix and permits PHP-CS-Fixer dry-run execution on that unsupported runtime.

Validation: PHP syntax checks and git diff --check passed; GitHub Actions will run the full suite.

@donatj
donatj marked this pull request as ready for review October 2, 2026 21:26
Copilot AI balanced review requested due to automatic review settings October 2, 2026 21:26

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.

Copilot review overview

🟢 Approval recommended

The focused compatibility changes correctly enable the new CI target without breaking older PHP-CS-Fixer versions.

Review effort: Balanced
Findings: None

What changed in this PR

Adds PHP 8.6 beta coverage while allowing PHP-CS-Fixer to run on unsupported runtimes.

Changes:

  • Adds PHP 8.6 to the CI matrix.
  • Enables PHP-CS-Fixer’s unsupported-version override when available.
File Description
.php-cs-fixer.dist.php Conditionally permits unsupported PHP runtimes.
.github/​workflows/​ci.yml Adds PHP 8.6 CI coverage.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants