Skip to content

ci: test against PHP 8.6 - #20

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

donatj wants to merge 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 this CI matrix currently does not exercise the upcoming release.

Adds PHP 8.6 to each PHP-version matrix while preserving quoted version values.

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

🔵 Needs a closer look

The unexplained Travis CI removal does not match the stated PR scope.

Review effort: Balanced
Findings: None

What changed in this PR

Adds PHP 8.6 coverage to the GitHub Actions CI matrix.

Changes:

  • Adds PHP 8.6 to GitHub Actions.
  • Removes the Travis CI configuration.
File Description
.travis.yml Removes legacy Travis CI configuration.
.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.

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