Skip to content

Commit c00be24

Browse files
committed
Tell dashboard LLM that CI failure alone isn't author/external
PRs can still be reviewed and approved while CI is failing, so CI failure on its own should not shift the bucket. Treat the boolean only as weak supporting evidence; the conversation drives the decision.
1 parent 39297bb commit c00be24

1 file changed

Lines changed: 7 additions & 3 deletions

File tree

.github/scripts/pull-request-dashboard.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,11 @@
5151
Decide who needs to act next on this PR using ONLY the context below. \
5252
Merge-conflict status is shown in a separate deterministic column of the \
5353
dashboard — do not infer it. CI is summarized as a single boolean \
54-
(failing yes/no); pending checks are treated as not-failing. Focus on \
55-
the conversation (comments, reviews, commits).
54+
(failing yes/no); pending checks are treated as not-failing. CI failure \
55+
on its own is NOT a reason to assign the PR to the author or to external: \
56+
PRs can still be reviewed and approved while CI is failing. Treat CI \
57+
status only as weak supporting evidence and focus on the conversation \
58+
(comments, reviews, commits).
5659
5760
Guidelines:
5861
- If the latest substantive activity is from the AUTHOR and there is an \
@@ -62,7 +65,8 @@
6265
- Use "external" ONLY when the conversation explicitly indicates the PR \
6366
is blocked on something outside this repo (e.g., an upstream PR, a \
6467
spec change, a release in another project). A new PR with no reviews \
65-
yet is NOT external — it is waiting on an approver.
68+
yet is NOT external — it is waiting on an approver. CI failing alone \
69+
is NOT external.
6670
6771
Respond with a single JSON object and nothing else (no prose, no fences):
6872
{{"side": "approver" | "author" | "external"}}

0 commit comments

Comments
 (0)