Skip to content

Repair wp/7.1 branch Actions: Remove obsolete and trunk-only workflows - #174

Closed
desrosj wants to merge 6 commits into
wp/7.1from
fix-actions-7.1/remove-rn-workflows
Closed

desrosj wants to merge 6 commits into
wp/7.1from
fix-actions-7.1/remove-rn-workflows

Conversation

@desrosj

@desrosj desrosj commented Sep 1, 2026 •

Copy link
Copy Markdown
Owner

What?

Removes every workflow whose triggers make it trunk-only, from wp/7.1.

Workflow Purpose Reason removed
check-backport-changelog.yml Verifies a Core backport changelog entry pull_request: branches: [trunk] only
check-package-changelogs.yml Verifies package CHANGELOG.md updates pull_request: branches: [trunk] only
stale-issue-gardening.yml Closes stale issues on a schedule GitHub only ever runs a scheduled workflow using the default branch's copy
sync-backport-changelog.yml Syncs Core backport changelog issues push: branches: [trunk], plus an issues: labeled trigger that — like the schedule case above — isn't tied to any git ref, so GitHub always runs it from the default branch's copy too
sync-assets-to-plugin-repo.yml Syncs assets/** to the WordPress.org plugin repo push: branches: [trunk] only
cherry-pick-wp-release.yml Auto cherry-picks labeled PRs to wp/X.Y branches Triggers only on pushes/labeled PRs against trunk; this branch is its destination, never its trigger source
storybook-pages.yml Deploys the canonical Storybook GitHub Pages site push: branches: [trunk] only
upload-release-to-plugin-repo.yml Publishes a Gutenberg plugin release to WordPress.org Triggers on release: types: [published]; in practice, releases are only ever published from release/X.Y branches, not from numbered wp/X.Y branches, so this branch is never the context a release actually happens in
pull-request-automation.yml Applies labels and milestones to pull requests, and comments a welcome/account-link prompt, before and after being merged Every job unconditionally checks out trunk for its actual logic, regardless of which branch triggered it

No React Native mobile workflows or gradle-wrapper-validation.yml exist on this branch at all — it was cut after those were already removed from trunk.

Why?

Keeping dead or inapplicable workflow files around adds maintenance surface (stale action pins, lint noise) with no corresponding benefit.

The workflows removed cannot be meaningfully executed against a branch other than trunk.

  • For schedule events, the event only ever fires for the default branch.
  • Most others are responsible for automation that is only relevant to upcoming plugin releases, which occur from trunk (except minor releases, which come from the respective release/X.Y 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.

@github-actions

github-actions Bot commented Sep 1, 2026 •

Copy link
Copy Markdown

Flaky tests detected in 9fd0b10.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/desrosj/gutenberg/actions/runs/34546322074
📝 Reported tests:

should update the HTML only for the latest navigation in /test/e2e/specs/interactivity/router-navigate.spec.ts, passed after 1 failed attempt.
TypeError: resolveLink2 is not a function
    at /home/runner/work/gutenberg/gutenberg/test/e2e/specs/interactivity/router-navigate.spec.ts:100:39
    at /home/runner/work/gutenberg/gutenberg/test/e2e/specs/interactivity/router-navigate.spec.ts:100:3

@desrosj desrosj changed the title Repair wp/7.1 branch Actions: Remove trunk-only workflows Repair wp/7.1 branch Actions: Remove obsolete and trunk-only workflows Sep 10, 2026
@desrosj
desrosj force-pushed the fix-actions-7.1/remove-rn-workflows branch from 82f3382 to 29682c7 Compare September 10, 2026 18:13
@desrosj
desrosj marked this pull request as ready for review September 10, 2026 18:32
@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 marked this pull request as draft September 10, 2026 20:11
@desrosj
desrosj force-pushed the fix-actions-7.1/remove-rn-workflows branch from 29682c7 to 327e271 Compare September 10, 2026 23:58
@desrosj
desrosj marked this pull request as ready for review September 11, 2026 00:25
@desrosj
desrosj marked this pull request as draft September 11, 2026 00:30
@desrosj
desrosj marked this pull request as ready for review September 11, 2026 00:39
@desrosj
desrosj marked this pull request as draft September 11, 2026 00:40
@desrosj
desrosj marked this pull request as ready for review September 11, 2026 00:41
@desrosj
desrosj force-pushed the fix-actions-7.1/remove-rn-workflows branch from 9fd0b10 to d11a0d1 Compare September 11, 2026 16:57
`storybook-pages.yml` deploys the single canonical Storybook GitHub
Pages site and only triggers on pushes to `trunk`; there is one
Storybook site for the whole project, not one per branch, so this
branch's copy can never run.
`cherry-pick-wp-release.yml` only triggers on pushes to `trunk` or
labeled pull requests targeting `trunk`. `wp/X.Y` branches, including
this one, are its destination, never its trigger source, so it can
never run here.

`publish-npm-packages.yml` is kept on this branch per updated project
policy.
…oad workflows.

`sync-assets-to-plugin-repo.yml` triggers only on `push: branches:
[trunk]` events that touch `assets/**`, so it can never run against
this branch. `upload-release-to-plugin-repo.yml` is removed too: it
triggers on `release: types: [published]`, which isn't literally
trunk-restricted, but no release events actually happen in the
context of these numbered `wp/X.Y` branches — releases are cut from
separate `release/X.Y` branches instead — so it's functionally dead
weight here despite the trigger being technically reachable.
`stale-issue-gardening.yml` is schedule-triggered, and GitHub only
ever runs a scheduled workflow using the copy on the repository's
default branch, regardless of any copy present elsewhere — this
branch's copy can never execute.

`sync-backport-changelog.yml` triggers on `push: branches: [trunk]`
or `issues: types: [labeled]`. The push trigger is trunk-only, and
the issue-label trigger isn't tied to any git ref at all — like the
schedule case above, GitHub always runs it using the workflow file
from the repository's default branch, so this branch's copy is never
the one that actually executes for that event either.
`check-backport-changelog.yml` and `check-package-changelogs.yml`
both trigger on `pull_request: branches: [trunk]` only, so neither
can ever run against a pull request targeting this branch.
@desrosj
desrosj force-pushed the fix-actions-7.1/remove-rn-workflows branch from d11a0d1 to b7b87c8 Compare September 11, 2026 18:15
@desrosj
desrosj removed this pull request from stack #177 September 11, 2026 18:31
@desrosj
desrosj added this pull request to stack #278 September 11, 2026 18:33
@github-actions

Copy link
Copy Markdown

🤖 PR meta 🤖

🏁 Flaky tests

Show the failures

Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

Should save the changes in /test/e2e/specs/editor/plugins/wp-editor-meta-box.spec.js, passed after 2 failed attempts.
TimeoutError: page.waitForFunction: Timeout 10000ms exceeded.
    at /home/runner/work/gutenberg/gutenberg/test/e2e/specs/editor/plugins/wp-editor-meta-box.spec.js:37:14
TimeoutError: locator.click: Timeout 10000ms exceeded.
Call log:
  - waiting for locator('role=button[name="Visual"i]')

    at /home/runner/work/gutenberg/gutenberg/test/e2e/specs/editor/plugins/wp-editor-meta-box.spec.js:36:55

b7b87c8 Run

@desrosj

desrosj commented Sep 11, 2026

Copy link
Copy Markdown
Owner Author

Migrated to real upstream: WordPress#82787. Closing this fork PR.

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