Skip to content

Repair wp/6.4 branch Actions: remove the mobile-unit-js job - #409

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

desrosj wants to merge 1 commit into
fix-actions-6.4/disable-flaky-test-issue-reportingfrom
fix-actions-6.4/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 as Mobile) from .github/workflows/unit-test.yml.

Nothing else in the file changes. The remaining four jobs — unit-js, test-php, phpcs and unit-php — are untouched, no job used needs: [mobile-unit-js], and no job's outputs: existed only to feed it, so the removal is self-contained.

Why?

The job runs npm run test:native, the Jest suite for the React Native mobile app. #109, at the bottom of this stack, already removed the rest of this branch's React Native CI surface — rnmobile-ios-runner.yml, rnmobile-android-runner.yml and gradle-wrapper-validation.yml — on the grounds that React Native mobile CI does not belong on a frozen release branch.

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 #402 September 21, 2026 19:29
@desrosj
desrosj removed this pull request from stack #402 September 21, 2026 21:08
@desrosj
desrosj changed the base branch from fix-actions-6.4/pin-wp-version-performance-tests to wp/6.4 September 21, 2026 21:10
@desrosj
desrosj force-pushed the fix-actions-6.4/remove-mobile-unit-js-job branch from 776f2ec to 916a198 Compare September 21, 2026 21:13
@desrosj
desrosj changed the base branch from wp/6.4 to fix-actions-6.4/pin-wp-version-performance-tests September 21, 2026 21:31
@desrosj
desrosj added this pull request to stack #423 September 21, 2026 21:32
@desrosj
desrosj removed this pull request from stack #423 September 21, 2026 21:48
@desrosj
desrosj changed the base branch from fix-actions-6.4/pin-wp-version-performance-tests to wp/6.4 September 21, 2026 21:50
@desrosj
desrosj force-pushed the fix-actions-6.4/remove-mobile-unit-js-job branch from 916a198 to 151343a Compare September 21, 2026 21:53
@desrosj
desrosj changed the base branch from wp/6.4 to fix-actions-6.4/pin-wp-version-performance-tests September 21, 2026 21:54
@desrosj
desrosj added this pull request to stack #428 September 21, 2026 21:54
@desrosj
desrosj removed this pull request from stack #428 September 21, 2026 23:17
@desrosj
desrosj changed the base branch from fix-actions-6.4/pin-wp-version-performance-tests to wp/6.4 September 21, 2026 23:18
@desrosj
desrosj force-pushed the fix-actions-6.4/remove-mobile-unit-js-job branch from 151343a to 5fb2cc7 Compare September 21, 2026 23:22
@desrosj
desrosj changed the base branch from wp/6.4 to fix-actions-6.4/pin-wp-version-performance-tests September 21, 2026 23:24
@desrosj
desrosj added this pull request to stack #432 September 21, 2026 23:24
@desrosj
desrosj removed this pull request from stack #432 September 21, 2026 23:49
@desrosj
desrosj changed the base branch from fix-actions-6.4/pin-wp-version-performance-tests to wp/6.4 September 21, 2026 23:50
@desrosj
desrosj force-pushed the fix-actions-6.4/remove-mobile-unit-js-job branch from 5fb2cc7 to dcf6bbf Compare September 21, 2026 23:51
@desrosj
desrosj changed the base branch from wp/6.4 to fix-actions-6.4/pin-wp-version-performance-tests September 21, 2026 23:53
@desrosj
desrosj added this pull request to stack #440 September 21, 2026 23:54
@desrosj
desrosj removed this pull request from stack #440 September 22, 2026 15:39
@desrosj
desrosj changed the base branch from fix-actions-6.4/pin-wp-version-performance-tests to wp/6.4 September 22, 2026 15:40
@desrosj
desrosj force-pushed the fix-actions-6.4/remove-mobile-unit-js-job branch from dcf6bbf to c8e1b64 Compare September 22, 2026 15:41
@desrosj
desrosj changed the base branch from wp/6.4 to fix-actions-6.4/disable-flaky-test-issue-reporting September 22, 2026 15:44
@desrosj
desrosj added this pull request to stack #458 September 22, 2026 15:44
The `mobile-unit-js` job (displayed as `Mobile`) runs
`npm run test:native`, the Jest suite for the React Native mobile app.
It is not disabled on this branch and it is not broken: it carries the
ordinary `if: ${{ github.repository == 'WordPress/gutenberg' ||
github.event_name == 'pull_request' }}` condition, runs on every pull
request against this stack, and currently passes in about five and a
half minutes.

It is removed for scope and consistency, not because it fails. #109
already removed `rnmobile-ios-runner.yml`, `rnmobile-android-runner.yml`
and `gradle-wrapper-validation.yml` from this branch on the grounds that
React Native mobile CI does not belong on a frozen release branch: the
mobile app is released from its own repository on its own schedule and
is never built from `wp/6.4`, so nothing here can regress it and no
result produced here is ever consulted. `mobile-unit-js` is the last
remaining piece of that same mobile CI surface, and leaving it behind
means this branch keeps paying for a runner on every pull request to
test a target it does not ship.

No other job references it. Nothing in the workflow uses `needs:
[mobile-unit-js]`, and no job's `outputs:` existed only to feed it, so
the removal is self-contained and the remaining four jobs (`unit-js`,
`test-php`, `phpcs`, `unit-php`) are untouched.

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 #458 September 22, 2026 20:15
@desrosj
desrosj changed the base branch from fix-actions-6.4/disable-flaky-test-issue-reporting to wp/6.4 September 22, 2026 20:16
@desrosj
desrosj force-pushed the fix-actions-6.4/remove-mobile-unit-js-job branch from c8e1b64 to dcb0d10 Compare September 22, 2026 21:31
@desrosj
desrosj changed the base branch from wp/6.4 to fix-actions-6.4/disable-flaky-test-issue-reporting September 22, 2026 21:34
@desrosj
desrosj added this pull request to stack #482 September 22, 2026 21:34
@desrosj
desrosj removed this pull request from stack #482 September 23, 2026 01:30
@desrosj
desrosj changed the base branch from fix-actions-6.4/disable-flaky-test-issue-reporting to wp/6.4 September 23, 2026 01:30
@desrosj
desrosj changed the base branch from wp/6.4 to fix-actions-6.4/disable-flaky-test-issue-reporting September 23, 2026 01:37
@desrosj
desrosj added this pull request to stack #485 September 23, 2026 01:38
@desrosj
desrosj removed this pull request from stack #485 September 24, 2026 04:44
@desrosj
desrosj changed the base branch from fix-actions-6.4/disable-flaky-test-issue-reporting to wp/6.4 September 24, 2026 04:45
@desrosj
desrosj changed the base branch from wp/6.4 to fix-actions-6.4/disable-flaky-test-issue-reporting September 24, 2026 04:47
@desrosj
desrosj added this pull request to stack #508 September 24, 2026 04:48
@desrosj
desrosj removed this pull request from stack #508 September 24, 2026 17:43
@desrosj
desrosj added this pull request to stack #514 September 24, 2026 17:44
@desrosj
desrosj marked this pull request as ready for review September 25, 2026 16:51
@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 #514 September 25, 2026 17:16
@desrosj
desrosj added this pull request to stack #533 September 25, 2026 17:16
@desrosj
desrosj removed this pull request from stack #533 September 28, 2026 15:23
@desrosj
desrosj added this pull request to stack #550 September 28, 2026 15:24
@desrosj
desrosj removed this pull request from stack #550 September 28, 2026 16:24
@desrosj
desrosj added this pull request to stack #555 September 28, 2026 16:25
@desrosj
desrosj removed this pull request from stack #555 September 28, 2026 18:59
@desrosj
desrosj added this pull request to stack #565 September 28, 2026 19:01
@desrosj

desrosj commented Sep 28, 2026

Copy link
Copy Markdown
Owner Author

Moved to WordPress#83674. The branch is left in place.

@desrosj desrosj closed this Sep 28, 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