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 |
|
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 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. |
baada98 to
d0f1557
Compare
|
Size Change: 0 B Total Size: 1.26 MB ℹ️ View Unchanged
|
d0f1557 to
fddb415
Compare
|
Size Change: 0 B Total Size: 1.26 MB ℹ️ View Unchanged
|
wp/6.1 branch Actions: remove the mobile-unit-js jobwp/6.1 branch Actions: Remove the mobile-unit-js job
fddb415 to
3b223ed
Compare
Part of #82932.
What?
Removes the
mobile-unit-jsjob (job nameMobile) from.github/workflows/unit-test.yml.The job runs the React Native editor's JavaScript unit tests —
npm run native test, which resolves tonpm run --prefix packages/react-native-editor test, i.e. Jest againsttest/native/jest.config.js.Why?
This job currently runs and passes on
wp/6.1. It is not disabled and it is not broken — it is being removed for scope and consistency, not to fix a failure. (Note for anyone comparing with the equivalentwp/7.0andwp/6.9PRs: on those branches the job is permanently disabled viaif: falsebecause React 19 broke the tests. That is not the situation here; onwp/6.1the job still carries the ordinaryif: ${{ github.repository == 'WordPress/gutenberg' || github.event_name == 'pull_request' }}condition.)The reason is React Native cleanup. React Native mobile support is no longer maintained or shipped as part of Gutenberg, and the first PR in this stack already removed the React Native workflows (
rnmobile-ios-runner.yml,rnmobile-android-runner.yml) from this branch as not belonging on a frozen release branch. This job is the last remaining piece of that test surface still running in CI onwp/6.1, and keeping it means every run of the required Unit Tests workflow spends a runner building and exercising code that this branch will never ship an update for.Nothing else in the workflow referenced the job — there are no
needs:entries naming it and nooutputs:that existed only to feed it — so the removal is self-contained. Theunit-js(JavaScript) andunit-php(PHP) jobs are untouched.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.
🤖 Generated with Claude Code