From edb13c777ba9051042847872c40489142aa2d627 Mon Sep 17 00:00:00 2001 From: Shelley Vohr Date: Fri, 1 Nov 2024 16:26:58 +0100 Subject: [PATCH] fix: allow unlabeling draft PRs --- src/api-review-state.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/api-review-state.ts b/src/api-review-state.ts index 29c2d22..7c28d48 100644 --- a/src/api-review-state.ts +++ b/src/api-review-state.ts @@ -463,8 +463,8 @@ export function setupAPIReviewStateManagement(probot: Probot) { throw new Error('Something went wrong - label does not exist.'); } - // Once a PR is merged, allow tampering. - if (pr.merged) return; + // It's permissible to change labels on merged or draft PRs. + if (pr.merged || pr.draft) return; // Exclude PRs from API review if they: // 1) Have backport, backport-skip, or fast-track labels