Skip to content

Repair wp/6.5 branch Actions: update test-php job's PHP version matrix - #267

Closed
desrosj wants to merge 1 commit into
fix-actions-6.5/fix-workflow-branch-filtersfrom
fix-actions-6.5/update-php-test-matrix
Closed

desrosj wants to merge 1 commit into
fix-actions-6.5/fix-workflow-branch-filtersfrom
fix-actions-6.5/update-php-test-matrix

Conversation

@desrosj

@desrosj desrosj commented Sep 11, 2026

Copy link
Copy Markdown
Owner

What?

Updates .github/workflows/unit-test.yml's test-php job's PHP version test matrix from 7.2, 7.3, 7.4, 8.0, 8.1, 8.2 to 7.0, 7.4, 8.0, 8.3.

Why?

Per WordPress core's PHP compatibility and WordPress versions policy, wp/6.5 supports PHP 7.0–7.4 and 8.0–8.3. This branch-repair project's testing convention is to test only the lowest and highest supported minor version within each supported major version, not every minor in between.

The previous matrix didn't follow this convention and also didn't match the actual supported range: it tested unnecessary intermediate versions (7.3, 8.1) while never testing the lowest supported minors (7.0, 7.1) and stopping one minor short of the highest supported 8.x release (8.2 instead of 8.3).

The corrected matrix is 7.0, 7.4, 8.0, 8.3 — the lowest and highest minor for each of the two supported majors.

Use of AI Tools

This PR was created by Claude Code under my supervision. All code should be treated as AI-produced and not yet reviewed by a human until this PR is marked Ready for Review.

@desrosj
desrosj added this pull request to stack #190 September 11, 2026 15:28
@desrosj
desrosj force-pushed the fix-actions-6.5/update-php-test-matrix branch from efc2497 to b125b09 Compare September 11, 2026 16:10
@desrosj
desrosj force-pushed the fix-actions-6.5/update-php-test-matrix branch from b125b09 to a58cd6f Compare September 13, 2026 01:30
@desrosj
desrosj force-pushed the fix-actions-6.5/update-php-test-matrix branch from a58cd6f to 7ebaeb1 Compare September 13, 2026 02:54
@desrosj
desrosj force-pushed the fix-actions-6.5/update-php-test-matrix branch from 7ebaeb1 to 316a179 Compare September 14, 2026 02:01
@desrosj
desrosj force-pushed the fix-actions-6.5/update-php-test-matrix branch from 316a179 to 4515e94 Compare September 14, 2026 17:38
@desrosj
desrosj marked this pull request as ready for review September 15, 2026 19:05
@github-actions

Copy link
Copy Markdown

Warning: Type of PR label mismatch

To merge this PR, it requires exactly 1 label indicating the type of PR. Other labels are optional and not being checked here.

  • Required label: Any label starting with [Type].
  • Labels found: .

Read more about Type labels in Gutenberg. Don't worry if you don't have the required permissions to add labels; the PR reviewer should be able to help with the task.

@github-actions

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: desrosj <desrosj@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@desrosj
desrosj force-pushed the fix-actions-6.5/update-php-test-matrix branch from 4515e94 to 25fe389 Compare September 15, 2026 19:23
@desrosj
desrosj removed this pull request from stack #190 September 15, 2026 21:18
The test-php job's matrix tested PHP 7.2, 7.3, 7.4, 8.0, 8.1, and 8.2 —
six versions, three per supported major. Per WordPress core's PHP
compatibility policy, wp/6.5 supports PHP 7.0 through 7.4 and 8.0
through 8.3. This project's testing convention covers only the lowest
and highest supported minor version within each supported major, not
every minor in between, so the matrix should have tested 7.0, 7.4,
8.0, and 8.3 instead.

The old matrix both tested unnecessary intermediate versions (7.3,
8.1) and missed the actual boundaries of the supported range: it never
tested PHP 7.0 or 7.1 at all, and stopped one minor short of the
newest supported 8.x release (8.2 instead of 8.3).

Updated `.github/workflows/unit-test.yml`'s `test-php` job matrix to
`7.0`, `7.4`, `8.0`, `8.3`.
@desrosj
desrosj force-pushed the fix-actions-6.5/update-php-test-matrix branch from 25fe389 to 02e1398 Compare September 15, 2026 21:19
@desrosj
desrosj changed the base branch from fix-actions-6.5/add-workflow-concurrency to fix-actions-6.5/fix-workflow-branch-filters September 15, 2026 21:19
@desrosj
desrosj added this pull request to stack #376 September 15, 2026 21:22
@desrosj

desrosj commented Sep 16, 2026

Copy link
Copy Markdown
Owner Author

Migrated to the real WordPress/gutenberg repository as WordPress#83000.

@desrosj desrosj closed this Sep 16, 2026
@desrosj
desrosj deleted the fix-actions-6.5/update-php-test-matrix branch October 29, 2026 22:05
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.

1 participant