Build/Test Tools: Trim the routine PHPUnit database matrix in the 6.8 branch - #13543
lancewillett wants to merge 1 commit into
Conversation
… branch. Test MariaDB 5.5, 10.6, 10.11, and 11.4 on pushes and pull requests. Dispatched runs, including the monthly run from test-old-branches.yml, keep the full database matrix. Scheduled runs only fire from the default branch, so on a release branch the dispatch is the equivalent of the weekly full run on trunk. Reduces the MySQL and MariaDB jobs on a pull request from 87 to 63. Follow-up to [63584]. See #66069. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
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 Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
desrosj
left a comment
There was a problem hiding this comment.
Same comment as on #13540
The changes themselves look fine. However, I find the new syntax to be pretty hard to read. I've opened #13547 to propose a different syntax to accomplish the same thing. Not a hill I am going to die on, but happy for you to at least consider this as a suggestion to improve readability.
And as well as the other PRs:
Is it intentional to not support the full matrix label in this branch?
Trac ticket: https://core.trac.wordpress.org/ticket/66069
Carries the routine MariaDB trim from r63584 to the 6.8 branch. The MySQL list already matches trunk's routine set.
Scheduled runs only fire from the default branch, so this keys the full list on
workflow_dispatchrather thanschedule.test-old-branches.ymldispatches this branch's PHPUnit workflow monthly, so the dropped versions keep running there.MySQL and MariaDB jobs on a pull request: 87 → 63. Dispatched runs: 87, unchanged.
Validation
pull_requestandworkflow_dispatch, before and after: 87 → 63 and 87 → 87.actionlintandzizmor --persona=regularreport the same findings before and after.Use of AI tools
AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Opus 5
Used for: Measuring job counts, the workflow edit, matrix validation, and drafting this description. Reviewed by me.
This pull request is for code review only. Please keep other discussion in the Trac ticket. Do not merge this pull request.