Skip to content

ci: test against PHP 8.6 - #20

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

🟡 Changes recommended

The unsupported-runtime exception also affects mutating local fixer runs and future PHP versions.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Adds PHP 8.6 beta coverage while allowing PHP-CS-Fixer to run on that runtime.

Changes:

  • Adds PHP 8.6 to the CI matrix.
  • Enables unsupported PHP versions in PHP-CS-Fixer configuration.
File Description
.php-cs-fixer.dist.php Allows fixer execution on unsupported 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.

Comment thread .php-cs-fixer.dist.php
Comment on lines +18 to +20
if( method_exists($config, 'setUnsupportedPhpVersionAllowed') ) {
$config->setUnsupportedPhpVersionAllowed(true);
}
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