Skip to content

Add PHP 8.5 to CI matrix#433

Merged
Alanaktion merged 2 commits intomasterfrom
copilot/add-php-8-5-workflows
Mar 5, 2026
Merged

Add PHP 8.5 to CI matrix#433
Alanaktion merged 2 commits intomasterfrom
copilot/add-php-8-5-workflows

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 5, 2026

Extends the CI test matrix to include PHP 8.5 alongside the existing 8.1–8.4 versions.

Changes

  • .github/workflows/ci.yml: Added "8.5" to the php matrix, adding 2 new job combinations (MySQL + SQLite)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Alanaktion <236490+Alanaktion@users.noreply.github.com>
Copilot AI changed the title [WIP] Add PHP 8.5 to the workflows Add PHP 8.5 to CI matrix Mar 5, 2026
@Alanaktion Alanaktion marked this pull request as ready for review March 5, 2026 22:35
Copilot AI review requested due to automatic review settings March 5, 2026 22:35
@Alanaktion Alanaktion merged commit 4f9f504 into master Mar 5, 2026
26 checks passed
@Alanaktion Alanaktion deleted the copilot/add-php-8-5-workflows branch March 5, 2026 22:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Extends the GitHub Actions CI workflow to run the existing test/install steps on PHP 8.5 in addition to the currently covered PHP 8.1–8.4 versions.

Changes:

  • Added PHP 8.5 to the CI matrix, expanding coverage across existing DB variants (MySQL + SQLite).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

strategy:
matrix:
php: ["8.1", "8.2", "8.3", "8.4"]
php: ["8.1", "8.2", "8.3", "8.4", "8.5"]
Copy link

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

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

Adding PHP 8.5 to the matrix will run the job on 8.5, but the Composer dependency resolution is still pinned to PHP 8.1.33 via composer.json's config.platform.php. That means the new 8.5 job won’t validate that dependencies/lockfile are resolvable under PHP 8.5 (it will install as if on 8.1). If the intent is to ensure full compatibility per PHP version, consider overriding/removing the platform PHP version for CI runs (or setting it dynamically from matrix.php).

Copilot uses AI. Check for mistakes.
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.

3 participants