Skip to content

Allow evaluating a PR's eligibility manually - #3406

Merged
pwizla merged 1 commit into
mainfrom
repo/automerge-eligibility-dispatch
Aug 26, 2026
Merged

Allow evaluating a PR's eligibility manually#3406
pwizla merged 1 commit into
mainfrom
repo/automerge-eligibility-dispatch

Conversation

@pwizla

@pwizla pwizla commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

The auto-merge eligibility workflow only listened to pull_request events, so an already-open PR could not be evaluated without editing it to trigger one. #3366 is exactly that case: a real pipeline PR that predates the workflow and has no way of being assessed.

  • workflow_dispatch now takes a pr_number input, and every reference to the PR number falls back to it
  • The five-minute quiet period is skipped by default on manual runs, since typing a PR number is not the same as pushing mid-burst
  • No Slack veto line is posted on manual runs: it would be noise for a PR already being looked at. The label is still set, so the cron behaves identically

Follows #3401

The workflow only ran on pull_request events, so an existing PR could not be
evaluated without editing it first. workflow_dispatch takes a PR number, skips
the quiet period since a manual run is already deliberate, and stays silent on
Slack because a veto line for a PR being looked at is noise.
@pwizla pwizla self-assigned this Aug 26, 2026
@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
documentation Ready Ready Preview Aug 26, 2026 2:08pm

Request Review

@github-actions github-actions Bot added pr: chore source: repo PRs/issues not targeting a specific documentation but rather affecting the whole repo internal PRs created by the Strapi core team labels Aug 26, 2026
@pwizla pwizla added this to the 7.1.1 milestone Aug 26, 2026
@pwizla

pwizla commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator Author

LGTM, let's merge so we can assess PR 3366 in prod

@pwizla
pwizla merged commit 02080c6 into main Aug 26, 2026
9 checks passed
@pwizla
pwizla deleted the repo/automerge-eligibility-dispatch branch August 26, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal PRs created by the Strapi core team pr: chore source: repo PRs/issues not targeting a specific documentation but rather affecting the whole repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant