Repository navigation
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Size Change: 0 B Total Size: 2.12 MB ℹ️ View Unchanged
|
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. Props Bot commandsYou can trigger Props Bot actions by commenting on this PR:
|
There was a problem hiding this comment.
🟡 Changes recommended
The create-block workflow unintentionally expands pull-request testing from Ubuntu only to all three operating systems.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Updates wp/6.9 CI to use its .nvmrc Node.js version and avoid unnecessary version checks.
Changes:
- Removes Node.js 22/24 test matrix entries.
- Removes
check-latestfrom Node.js setup steps. - Simplifies job names.
File summaries
| File | Description |
|---|---|
.github/workflows/unit-test.yml |
Uses the branch’s default Node.js version. |
.github/workflows/static-checks.yml |
Removes latest-version checks. |
.github/workflows/publish-npm-packages.yml |
Removes latest-version checks during publishing. |
.github/workflows/create-block.yml |
Removes the Node.js matrix. |
.github/workflows/bundle-size.yml |
Removes latest-version checks. |
.github/workflows/build-plugin-zip.yml |
Removes latest-version checks from build and publishing jobs. |
.github/setup-node/action.yml |
Stops checking for newer matching Node.js releases. |
Review details
- Files reviewed: 7/7 changed files
- Comments generated: 1
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
300707c to
cd826dc
Compare
cd826dc to
3a9fb3a
Compare
3a9fb3a to
7cba425
Compare
7cba425 to
870d0bf
Compare
870d0bf to
9b2a410
Compare
9b2a410 to
0ff4d1d
Compare
0ff4d1d to
72c46b4
Compare
72c46b4 to
2b75865
Compare
2b75865 to
084f04d
Compare
wp/6.9 branch Actions: Test only the Node.js version this branch supportswp/6.9 branch Actions: Test against a single Node.js version
084f04d to
711bbe8
Compare
Part of #82785.
What?
This trims the testing matrix for workflow files that test multiple versions of Node.js.
Additionally, the
check-latest: trueinput is removed from everyactions/setup-nodecall.create-block.yml's pull requestexcludeentries for macOS and Windows are kept; only thenodeones are removed.Why?
These workflows tested Node.js 20, 22 and 24 when the branch was created to catch forward-compatibility issues early. Since this branch is only eligible for security updates, it will never run on anything but the Node.js version pinned in
.nvmrc. Testing additional versions wastes time running Actions unnecessarily.The
check-latest: trueinput makesactions/setup-nodere-check for a newer matching release on every single run, useful only while the pinned version is still receiving releases. Node.js 20.x has reached end-of-life.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.