Skip to content

Repair wp/6.2 branch Actions: remove the mobile-unit-js job - #407

Closed
desrosj wants to merge 1 commit into
fix-actions-6.2/disable-flaky-test-issue-reportingfrom
fix-actions-6.2/remove-mobile-unit-js-job
Closed

desrosj wants to merge 1 commit into
fix-actions-6.2/disable-flaky-test-issue-reportingfrom
fix-actions-6.2/remove-mobile-unit-js-job

Conversation

@desrosj

@desrosj desrosj commented Sep 21, 2026 •

Copy link
Copy Markdown
Owner

What?

Removes the mobile-unit-js job (displayed in CI as Mobile) from .github/workflows/unit-test.yml.

That job runs npm run native test, which resolves to npm run --prefix packages/react-native-editor test — the React Native editor package's own Jest suite, run against test/native/jest.config.js. In other words, it is the React Native mobile JavaScript unit test run.

Nothing else in the workflow referred to it: no needs: entry named mobile-unit-js, and no job produced outputs: that existed only to feed it. Removing the job block is the entire change — the remaining jobs (unit-js, test-php, phpcs, and the deprecated unit-php aggregator, which only needs test-php and phpcs) are untouched.

Why?

This is the last piece of React Native mobile CI left on this branch.

The bottom of this stack already removed the other React Native workflows — rnmobile-ios-runner.yml, rnmobile-android-runner.yml, and gradle-wrapper-validation.yml — on the grounds that React Native mobile support is no longer maintained or shipped by Gutenberg, so CI dedicated to exercising it serves no purpose on a frozen wp/6.2 maintenance branch. The mobile-unit-js job is the same category of work: it tests the React Native editor, just at the unit level and on a Linux runner rather than through the native iOS/Android runners. Leaving it behind means this branch still spends CI time on mobile code it no longer maintains, which is exactly the surface that cleanup set out to retire.

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

@desrosj
desrosj added this pull request to stack #350 September 21, 2026 19:29
@desrosj
desrosj removed this pull request from stack #350 September 21, 2026 21:45
@desrosj
desrosj changed the base branch from fix-actions-6.2/pin-wp-version-performance-tests to wp/6.2 September 21, 2026 21:46
@desrosj
desrosj force-pushed the fix-actions-6.2/remove-mobile-unit-js-job branch from 4873f47 to 7e5f38a Compare September 21, 2026 21:50
@desrosj
desrosj changed the base branch from wp/6.2 to fix-actions-6.2/pin-wp-version-performance-tests September 21, 2026 21:51
@desrosj
desrosj added this pull request to stack #427 September 21, 2026 21:51
@desrosj
desrosj removed this pull request from stack #427 September 21, 2026 23:47
@desrosj
desrosj changed the base branch from fix-actions-6.2/pin-wp-version-performance-tests to wp/6.2 September 21, 2026 23:48
@desrosj
desrosj force-pushed the fix-actions-6.2/remove-mobile-unit-js-job branch from 7e5f38a to 4e0d993 Compare September 21, 2026 23:49
@desrosj
desrosj changed the base branch from wp/6.2 to fix-actions-6.2/pin-wp-version-performance-tests September 21, 2026 23:51
@desrosj
desrosj added this pull request to stack #435 September 21, 2026 23:51
@desrosj
desrosj removed this pull request from stack #435 September 22, 2026 15:49
@desrosj
desrosj changed the base branch from fix-actions-6.2/pin-wp-version-performance-tests to wp/6.2 September 22, 2026 15:50
@desrosj
desrosj changed the base branch from wp/6.2 to fix-actions-6.2/disable-flaky-test-issue-reporting September 22, 2026 15:53
@desrosj
desrosj force-pushed the fix-actions-6.2/remove-mobile-unit-js-job branch from 4e0d993 to 0dc91a7 Compare September 22, 2026 15:53
@desrosj
desrosj added this pull request to stack #460 September 22, 2026 15:53
@desrosj
desrosj removed this pull request from stack #460 September 22, 2026 20:20
@desrosj
desrosj changed the base branch from fix-actions-6.2/disable-flaky-test-issue-reporting to wp/6.2 September 22, 2026 20:20
@desrosj
desrosj force-pushed the fix-actions-6.2/remove-mobile-unit-js-job branch from 0dc91a7 to f6e8b32 Compare September 22, 2026 20:22
@desrosj
desrosj changed the base branch from wp/6.2 to fix-actions-6.2/disable-flaky-test-issue-reporting September 22, 2026 20:36
@desrosj
desrosj added this pull request to stack #478 September 22, 2026 20:36
@desrosj
desrosj removed this pull request from stack #478 September 23, 2026 01:31
@desrosj
desrosj changed the base branch from fix-actions-6.2/disable-flaky-test-issue-reporting to wp/6.2 September 23, 2026 01:31
@desrosj
desrosj changed the base branch from wp/6.2 to fix-actions-6.2/disable-flaky-test-issue-reporting September 23, 2026 01:44
@desrosj
desrosj added this pull request to stack #487 September 23, 2026 01:44
@desrosj
desrosj removed this pull request from stack #487 September 24, 2026 17:15
@desrosj
desrosj changed the base branch from fix-actions-6.2/disable-flaky-test-issue-reporting to wp/6.2 September 24, 2026 17:15
@desrosj
desrosj changed the base branch from wp/6.2 to fix-actions-6.2/disable-flaky-test-issue-reporting September 24, 2026 17:17
@desrosj
desrosj added this pull request to stack #513 September 24, 2026 17:18
@desrosj
desrosj removed this pull request from stack #513 September 24, 2026 17:45
@desrosj
desrosj added this pull request to stack #516 September 24, 2026 17:46
@desrosj
desrosj removed this pull request from stack #516 September 28, 2026 14:33
@desrosj
desrosj added this pull request to stack #539 September 28, 2026 14:34
@desrosj
desrosj removed this pull request from stack #539 September 28, 2026 19:05
@desrosj
desrosj changed the base branch from fix-actions-6.2/disable-flaky-test-issue-reporting to wp/6.2 September 28, 2026 19:05
@desrosj
desrosj changed the base branch from wp/6.2 to fix-actions-6.2/disable-flaky-test-issue-reporting September 28, 2026 19:06
@desrosj
desrosj added this pull request to stack #567 September 28, 2026 19:07
@desrosj
desrosj removed this pull request from stack #567 September 28, 2026 19:20
@desrosj
desrosj added this pull request to stack #572 September 28, 2026 19:21
@desrosj
desrosj marked this pull request as ready for review September 28, 2026 19:24
@desrosj
desrosj removed this pull request from stack #572 September 29, 2026 00:24
@desrosj
desrosj changed the base branch from fix-actions-6.2/disable-flaky-test-issue-reporting to wp/6.2 September 29, 2026 00:24
@desrosj
desrosj changed the base branch from wp/6.2 to fix-actions-6.2/disable-flaky-test-issue-reporting September 29, 2026 00:25
@desrosj
desrosj added this pull request to stack #573 September 29, 2026 00:26
@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 removed this pull request from stack #573 October 6, 2026 23:39
@desrosj
desrosj added this pull request to stack #637 October 6, 2026 23:44
The `mobile-unit-js` job (displayed as `Mobile`) runs `npm run native test`,
which delegates to `packages/react-native-editor`'s own Jest suite
(`test/native/jest.config.js`) — the React Native editor's JavaScript unit
tests.

This branch has already removed the rest of the React Native mobile CI:
`rnmobile-ios-runner.yml`, `rnmobile-android-runner.yml`, and
`gradle-wrapper-validation.yml` are all gone. React Native mobile support is
no longer maintained or shipped by Gutenberg, and testing it is not something
a frozen `wp/6.2` maintenance branch needs to keep doing. This job is the last
piece of that CI surface still present here.

Unlike `wp/6.9` and `wp/7.0`, where the equivalent job is permanently disabled
with `if: false`, the job on this branch is not disabled: it carries the
ordinary `if: ${{ github.repository == 'WordPress/gutenberg' || github.event_name == 'pull_request' }}`
condition, it runs, and it passes. It is removed for scope and consistency
with the React Native cleanup already done on this branch, not because it is
broken.

Nothing else in the workflow referenced it — no `needs:` entry named it, and
no job produced outputs solely for it — so removing the job block is the whole
change.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DtSAAJkQ9P6vPtDumqStg8

# Conflicts:
#	.github/workflows/unit-test.yml
@desrosj
desrosj removed this pull request from stack #637 October 7, 2026 17:51
@desrosj
desrosj changed the base branch from fix-actions-6.2/disable-flaky-test-issue-reporting to wp/6.2 October 7, 2026 17:52
@desrosj
desrosj force-pushed the fix-actions-6.2/remove-mobile-unit-js-job branch from f6e8b32 to b884da2 Compare October 7, 2026 17:54
@desrosj
desrosj changed the base branch from wp/6.2 to fix-actions-6.2/disable-flaky-test-issue-reporting October 7, 2026 17:55
@desrosj
desrosj added this pull request to stack #666 October 7, 2026 17:56
@desrosj

desrosj commented Oct 8, 2026

Copy link
Copy Markdown
Owner Author

Moved to WordPress#84221.

@desrosj desrosj closed this Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant