From c9225057c92ecb6084cad809239aa940f0e2f047 Mon Sep 17 00:00:00 2001 From: Michael Daniels Date: Sun, 25 Jan 2026 19:10:17 -0500 Subject: [PATCH] ci/github-script/check-target-branch: do not "Request changes" You can technically have multiple reviews requesting changes, but there's no good way to dismiss all of the reviews requesting changes from the same user using the UI. This makes minimization impossible (because all but one of the reviews is not dismissed, even though the PR is no longer blocked due to the review in GitHub's system). As a workaround, we will only comment. CI will still fail when appropriate. --- ci/github-script/check-target-branch.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ci/github-script/check-target-branch.js b/ci/github-script/check-target-branch.js index 339968e6a0bb..47e36b9cb3f6 100644 --- a/ci/github-script/check-target-branch.js +++ b/ci/github-script/check-target-branch.js @@ -137,7 +137,7 @@ async function checkTargetBranch({ github, context, core, dry }) { core, dry, body, - event: 'REQUEST_CHANGES', + event: 'COMMENT', reviewKey, }) @@ -165,7 +165,7 @@ async function checkTargetBranch({ github, context, core, dry }) { core, dry, body, - event: 'REQUEST_CHANGES', + event: 'COMMENT', reviewKey, })