Skip to content

ci: test against PHP 8.6 - #17

Merged
donatj merged 2 commits into
masterfrom
ci/add-php-8-6
Oct 5, 2026
Merged

donatj merged 2 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, so the primary CI matrix currently does not exercise the upcoming release.

Adds PHP 8.6 to the primary CI matrix while retaining the dedicated coverage workflow at its established PHP version.

Validation: YAML parsed successfully and git diff --check passed.

@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 supported setup action handles PHP 8.6, and the focused matrix update is correct.

Review effort: Balanced
Findings: None

What changed in this PR

Adds upcoming PHP 8.6 compatibility testing to primary CI.

Changes:

  • Extends the PHP test matrix through PHP 8.6.
  • Leaves coverage configuration unchanged.
File Description
.github/​workflows/​ci.yml Adds PHP 8.6 to the test matrix.

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

@donatj
donatj merged commit 8f26683 into master Oct 5, 2026
20 checks passed
@donatj
donatj deleted the ci/add-php-8-6 branch October 5, 2026 20:40
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