From d4cd168ffc6633117f36320655d4afc34f054a15 Mon Sep 17 00:00:00 2001 From: apiraino Date: Tue, 2 Sep 2025 09:07:51 +0200 Subject: [PATCH] Exclude drafts from auto-backport nominate --- src/handlers/backport.rs | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/src/handlers/backport.rs b/src/handlers/backport.rs index 18104eacc..b206d68ed 100644 --- a/src/handlers/backport.rs +++ b/src/handlers/backport.rs @@ -49,13 +49,20 @@ pub(super) async fn parse_input( return Ok(None); }; - // Only handle events when the PR is opened or the first comment is edited - let should_check = matches!(event.action, IssuesAction::Opened | IssuesAction::Edited); - if !should_check || !event.issue.is_pr() { + // Only handle the event when the PR: + // - is opened (and not a draft) + // - is converted from draft to ready for review + // - when the first comment is edited + let skip_check = !matches!( + event.action, + IssuesAction::Opened | IssuesAction::Edited | IssuesAction::ReadyForReview + ); + if skip_check || !event.issue.is_pr() || event.issue.draft { log::debug!( - "Skipping backport event because: IssuesAction = {:?} issue.is_pr() {}", + "Skipping backport event because: IssuesAction = {:?}, issue.is_pr() {}, draft = {}", event.action, - event.issue.is_pr() + event.issue.is_pr(), + event.issue.draft ); return Ok(None); }