Skip to content

Trigger PR triage on ready_for_review#22089

Open
BagToad wants to merge 2 commits into
developmentfrom
kw/triage-fix-ready-for-review
Open

Trigger PR triage on ready_for_review#22089
BagToad wants to merge 2 commits into
developmentfrom
kw/triage-fix-ready-for-review

Conversation

@BagToad

@BagToad BagToad commented May 8, 2026

Copy link
Copy Markdown
Member

PRs opened as draft and later marked ready for review were skipped by the triage workflow because ready_for_review was missing from both the workflow trigger list and the pr-requirements job filter. Adds it to both so screening runs when a draft PR transitions to requesting review.

Pairs with desktop/gh-cli-and-desktop-shared-workflows#18, which fixes the called workflow's own internal filters.

PRs opened as draft and later marked ready for review were skipped by
the triage workflow because ready_for_review was missing from both the
workflow trigger list and the pr-requirements job filter. Add it to
both so screening runs when a draft PR transitions to requesting
review.

Pairs with desktop/gh-cli-and-desktop-shared-workflows#18, which adds
ready_for_review to the called workflow's own internal filters.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the PR triage GitHub Actions workflow so that PR screening runs when a draft PR is marked Ready for review, ensuring those PRs aren’t skipped by automation.

Changes:

  • Adds ready_for_review to the pull_request_target event trigger types.
  • Extends the pr-requirements job condition to run on ready_for_review.
Show a summary per file
File Description
.github/workflows/triage-prs.yml Triggers triage and PR requirements screening when a draft PR transitions to ready-for-review.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

tidy-dev
tidy-dev previously approved these changes May 8, 2026
@niik

niik commented Jun 5, 2026

Copy link
Copy Markdown
Member

Couldn't merge this due to a lint error. If you could approve it again @tidy-dev then we're all set

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants