Skip to content

Repair wp/7.1 branch Actions: Reference trunk's reusable workflow-lint job - #176

Closed
desrosj wants to merge 1 commit into
fix-actions-7.1/remove-rn-workflowsfrom
fix-actions-7.1/reference-trunk-workflow-lint
Closed

desrosj wants to merge 1 commit into
fix-actions-7.1/remove-rn-workflowsfrom
fix-actions-7.1/reference-trunk-workflow-lint

Conversation

@desrosj

@desrosj desrosj commented Sep 1, 2026 •

Copy link
Copy Markdown
Owner

What?

Why?

Linting workflow files helps prevent incorrect and/or insecure patterns from being introduced into the codebase.

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 ea4ee12.
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/33516742390
📝 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 force-pushed the fix-actions-7.1/reference-trunk-workflow-lint branch 2 times, most recently from 8d8f5de to 2bf536c Compare September 10, 2026 20:10
actions: read
contents: read
uses: ./.github/workflows/reusable-workflow-lint.yml
uses: WordPress/gutenberg/.github/workflows/reusable-workflow-lint.yml@trunk
@desrosj
desrosj force-pushed the fix-actions-7.1/reference-trunk-workflow-lint branch from 2bf536c to 8426d2b Compare September 10, 2026 23:58
@desrosj
desrosj marked this pull request as ready for review September 11, 2026 00:35
@desrosj
desrosj marked this pull request as draft September 11, 2026 00:37
@desrosj
desrosj marked this pull request as ready for review September 11, 2026 00:41
@desrosj
desrosj force-pushed the fix-actions-7.1/reference-trunk-workflow-lint branch from 8426d2b to 806dd0d Compare September 11, 2026 00:46
@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 force-pushed the fix-actions-7.1/reference-trunk-workflow-lint branch from 806dd0d to 657a6c7 Compare September 11, 2026 16:58
Removes the local `reusable-workflow-lint.yml` and points
`workflow-lint.yml` at
`WordPress/gutenberg/.github/workflows/reusable-workflow-lint.yml@trunk`
instead, so this branch always lints against trunk's current lint
logic rather than a stale local copy.

Also widens `workflow-lint.yml`'s own `push`/`pull_request` branch
filters from `trunk` only to also include `release/[0-9]+.[0-9]`,
`wp/[0-9]+.[0-9]`, `wp/latest`, and `wp/next` — porting
`WordPress#82744`'s fix for this file. Without this, pushes
or pull requests targeting this branch itself never triggered linting
at all, since `trunk` was the only branch the filter recognized.
@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
@desrosj
desrosj force-pushed the fix-actions-7.1/reference-trunk-workflow-lint branch from 657a6c7 to 7787e94 Compare September 11, 2026 19:11
@desrosj

desrosj commented Sep 11, 2026

Copy link
Copy Markdown
Owner Author

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

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

7787e94 Run

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.

2 participants