Skip to content

CI: add PHP 8.5 to test matrix#29

Merged
LordSimal merged 2 commits into
2.xfrom
copilot/add-php-8-5-to-ci-matrix
Apr 21, 2026
Merged

CI: add PHP 8.5 to test matrix#29
LordSimal merged 2 commits into
2.xfrom
copilot/add-php-8-5-to-ci-matrix

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 21, 2026

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

Changes

  • .github/workflows/ci.yml: adds '8.5' to the php-version matrix in testsuite-linux

No composer.json changes needed — the existing "php": ">=8.1" constraint already covers 8.5.

Original prompt

Create a PR that updates the CI (GitHub Actions workflows) to add PHP 8.5 to the test matrix.

Repository: LordSimal/cakephp-dump-sql

Requirements:

  • Find the existing CI workflow(s) that run PHP tests (likely under .github/workflows/).
  • Add PHP 8.5 to the PHP version matrix (or equivalent) alongside existing supported versions.
  • Ensure the setup step uses shivammathur/setup-php (or the repo’s existing setup method) and that PHP 8.5 is included.
  • If there are version constraints in composer.json/composer.lock that would prevent running on PHP 8.5, update constraints appropriately only if required for CI to run.
  • Keep changes minimal and focused to CI.
  • Update any README/support matrix only if the repo already documents supported PHP versions.

Validation:

  • Ensure workflows remain valid YAML.
  • Ensure the matrix includes php: '8.5' (quoted) and existing versions unchanged.

Output:

  • Open a pull request with a clear title like "CI: add PHP 8.5" and describe what changed.

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: make PR's for all the following repos where it adds PHP 8.5 to the CI

@LordSimal/cakephp-dbml
@LordSimal/custom-html-elements
@LordSimal/cakephp-scheduler
@LordSimal/cakephp-dump-sql
@LordSimal/cakephp-sentry

Assistant: I’m waiting on your approval in the confirmation dialog(s) to proceed with creating the pull requests across those repositories.

User: @copilot Accepted Confirmation: Confirm agent session

Assistant: I’ve started working on the PR for LordSimal/cakephp-dbml. You can track progress at the task link below.

data:
  - title: "[WIP] CI: add PHP 8.5"
    repository: LordSimal/cakephp-dbml
    url: https://github.com/copilot/tasks/pull/PR_kwDOHBU7BM7UWa05?session_id=049c79fe-9cdd-440a-857b-57384fe3d5db
    state: queued

User: @copilot Accepted Confirmation: Confirm agent session

Assistant: I’m not able to continue creating additional PRs in this message.

So far:

  • LordSimal/cakephp-dbml: PR work has been started (you’ll see it in the agent task you already accepted).

Please send one more message (anything like “continue”) and I’ll kick off the remaining repos:

  • LordSimal/custom-html-elements
  • LordSimal/cakephp-scheduler
  • LordSimal/cakephp-dump-sql
  • LordSimal/cakephp-sentry

User: @copilot Accepted Confirmation: Confirm agent session

Assistant: I’ve started working on the PR for LordSimal/cakephp-scheduler. You can track progress at the task link below.

data:
  - title: "[WIP] CI: add PHP 8.5"
    repository: LordSimal/cakephp-scheduler
    url: https://github.com/copilot/tasks/pull/PR_kwDOJ5RT587UWcEa?session_id=b9e52c70-678e-4ac4-817a-2e81f3bc2596
    state: queued

This pull request was created from Copilot chat.

Agent-Logs-Url: https://github.com/LordSimal/cakephp-dump-sql/sessions/0aa2bb48-0db3-4de3-81b9-7f20194c5871

Co-authored-by: LordSimal <9105243+LordSimal@users.noreply.github.com>
Copilot AI changed the title [WIP] CI: add PHP 8.5 CI: add PHP 8.5 to test matrix Apr 21, 2026
Copilot AI requested a review from LordSimal April 21, 2026 15:28
@LordSimal LordSimal marked this pull request as ready for review April 21, 2026 15:32
@LordSimal LordSimal merged commit 90492bd into 2.x Apr 21, 2026
23 checks passed
@LordSimal LordSimal deleted the copilot/add-php-8-5-to-ci-matrix branch April 21, 2026 15:33
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