Skip to content

6.8 Branch: Build/Test Tools: Skip pull_request triggered runs in private repos. - #12992

Open
desrosj wants to merge 4 commits into
WordPress:6.8from
desrosj:backport/6.8-branch-63183
Open

desrosj wants to merge 4 commits into
WordPress:6.8from
desrosj:backport/6.8-branch-63183

Conversation

@desrosj

@desrosj desrosj commented Aug 12, 2026 •

Copy link
Copy Markdown
Member

Backports r63183 to the 6.8 branch.

In private forks and mirrors, pull_request workflows no longer run for draft PRs unless the Draft Workflow Runs label is added. The condition is applied to each job on this branch that runs for pull_request events outside of WordPress/wordpress-develop. Workflows from the trunk commit that don't exist on this branch are skipped.

Differences from trunk:

  • uses: references in upgrade-develop-testing.yml and workflow-lint.yml still point to @trunk. This branch has no local reusable workflows.
  • Existing github.actor != 'dependabot[bot]' checks are kept.
  • upgrade-testing.yml and local-docker-environment.yml (not gated this way on trunk) get the same condition.

Trac ticket: https://core.trac.wordpress.org/ticket/65845

Use of AI Tools

This pull request was created by an AI agent (Claude Code). Until this PR is marked "Ready for Review", treat it as untrusted, AI-created code that requires a manual review by a human team member.

🤖 Generated with Claude Code

This modifies the conditions for GitHub Actions workflow runs so that they do not run within private forks and mirrors for `pull_request` events when the PR is in `draft` status.

For situations where this may be desirable, a `Draft Workflow Runs` label can be added to a `draft` pull request to force workflows to run.

Props desrosj, jorbin.
See #65848.

git-svn-id: https://develop.svn.wordpress.org/trunk@63183 602fd350-edb4-49c9-b593-d223f7449a82
@desrosj desrosj self-assigned this Aug 12, 2026
@github-actions

Copy link
Copy Markdown

Test using WordPress Playground

The changes in this pull request can previewed and tested using a WordPress Playground instance.

WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser.

Some things to be aware of

  • All changes will be lost when closing a tab with a Playground instance.
  • All changes will be lost when refreshing the page.
  • A fresh instance is created each time the link below is clicked.
  • Every time this pull request is updated, a new ZIP file containing all changes is created. If changes are not reflected in the Playground instance,
    it's possible that the most recent build failed, or has not completed. Check the list of workflow runs to be sure.

For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation.

Test this pull request with WordPress Playground.

desrosj and others added 2 commits November 5, 2026 18:05
…orkflows.

Applies the same `pull_request` condition from r63183 to the jobs in `upgrade-testing.yml` and `local-docker-environment.yml`, which are still gated on `pull_request` on this branch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@desrosj
desrosj requested a review from a team November 5, 2026 23:13
@desrosj desrosj changed the title Backport: Skip pull_request triggered runs in private repos - 6.8 Branch 6.8 Branch: Build/Test Tools: Skip pull_request triggered runs in private repos. Nov 5, 2026
@desrosj desrosj moved this from Backlog to In review in WordPress Project Build Tooling Nov 5, 2026
@desrosj
desrosj marked this pull request as ready for review November 6, 2026 00:40
@github-actions

github-actions Bot commented Nov 6, 2026

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.

Core Committers: Use this line as a base for the props when committing in SVN:

Props desrosj.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.


Props Bot commands

You can trigger Props Bot actions by commenting on this PR:

  • @propsbot add <username> to give props to a GitHub user. Requires write access.
  • @propsbot remove <username> to leave a user out of the props. Requires write access.
  • @propsbot refresh to refresh this list. Adding the props-bot label works too.

@lancewillett lancewillett left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Compared with r63183 and the 6.8 branch workflows: every pull_request job on this branch now has the same private-repo, draft, and Draft Workflow Runs label condition as trunk (22 jobs across 11 files), and nothing else changes. Workflows trunk touched that this branch lacks are correctly left out. The existing dependabot check is kept, and upgrade-testing.yml and local-docker-environment.yml get the same condition since they run on pull_request here. It merges cleanly into the current 6.8 tip (2717ac1). CI: 214 passed, 1 failed. The failure is Local Docker Environment, PHP 8.4 / MySQL 5.7 (https://github.com/WordPress/wordpress-develop/actions/runs/42127965400/job/127597137170): npm ci died with EEXIST on a file in the npm cache (_cacache/tmp), before any test ran. The same job passes on the 6.8 branch (run 40721457115). I re-ran the failed job and it passed.

AI review · claude-opus-5-5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

2 participants