Add review checklists workflow - #899

Open

LittleHuba wants to merge 30 commits into

mainfrom

add-review-checklists-workflow

Commits on Aug 13, 2026

Add review checklists workflow and action

LittleHuba
and
Copilot
committed
4ebc05a
Split review-checklists workflow into unprivileged trigger + privileged apply stages

LittleHuba
and
Copilot
committed
d16ae4a
Minimize stage1->stage2 data transfer in review-checklists split

LittleHuba
and
Copilot
committed
93d72aa
Fix PR-number resolution: gh api --jq does not accept --arg

LittleHuba
and
Copilot
committed
e8fdce8
Drop stage1->stage2 action transfer; dismiss_sync self-determines no-op

LittleHuba
and
Copilot
committed
59c8a2c
Resolve PR number by head branch, not head SHA, to avoid a fast-iteration race

LittleHuba
and
Copilot
committed
f053622
Disable the example checklist
LittleHuba
committed
2079a19
Move review-checklists action to tools/review-checklists; add README

LittleHuba
and
Copilot
committed
51655eb
Post checklist findings as file-level comments to support binary files

LittleHuba
and
Copilot
committed
2a41567
Restrict compile_pip_requirements to linux
LittleHuba
committed
d500ea9
Validate checklist evidence in merge queue instead of assuming it

LittleHuba
and
Copilot
committed
2a211a7
Transfer PR number from trigger to apply stage via artifact

LittleHuba
and
Copilot
committed
0376227
Fix merge_group evidence validation never actually running

LittleHuba
and
Copilot
committed
1a6d2b6
Switch checklist path matching from fnmatch to pathspec (gitignore-style)

LittleHuba
and
Copilot
committed
1a2f61e
Reduce visibility
LittleHuba
committed
00a6d56
Fix stale copyright-checker srcs entry after actions/ -> tools/ move

LittleHuba
and
Copilot
committed
718544c
Add cancel-in-progress to review-checklists apply workflow concurrency

LittleHuba
and
Copilot
committed
8fe99cc
Document why review-checklists action installs deps with pip, not Bazel

LittleHuba
and
Copilot
committed
cc00f47
Clarify Otterdog status-check syntax for the review-checklists status

LittleHuba
and
Copilot
committed
7c6f5db
Deduplicate OK-reply lookup across review-checklists scripts

LittleHuba
and
Copilot
committed
0c83f34
Use Bzlmod module name instead of hardcoded '_main' for Rlocation

LittleHuba
and
Copilot
committed
2321b91
Document why get_approving_reviewers' discard call is not redundant

LittleHuba
and
Copilot
committed
505854c
Name the commit-status description length magic number

LittleHuba
and
Copilot
committed
ae028bf
Document why is_pr_in_merge_queue fails closed to False on error

LittleHuba
and
Copilot
committed
d57d3de
Stay pending until every relevant checklist has been posted

LittleHuba
and
Copilot
committed
8a2797d
Clarify why the merge-queue notice check is separate from merge_group handling

LittleHuba
and
Copilot
committed
11bda2d
Avoid single/two-letter abbreviations for checklist variables

LittleHuba
and
Copilot
committed
538322f
Improve remaining unclear variable names

LittleHuba
and
Copilot
committed
9e572e7
Rename relevant_ids/affected_ids to posted_relevant_ids/posted_affected_ids

LittleHuba
and
Copilot
committed
5d1be51
Rename relevant/affected checklist-list variables for clarity

LittleHuba
and
Copilot
committed
71a0974

Navigation Menu

Uh oh!

Add review checklists workflow - #899

Add review checklists workflow#899
LittleHuba wants to merge 30 commits into
mainfrom
add-review-checklists-workflow

Commits on Aug 13, 2026

Add review checklists workflow and action

Split review-checklists workflow into unprivileged trigger + privileged apply stages

Minimize stage1->stage2 data transfer in review-checklists split

Fix PR-number resolution: gh api --jq does not accept --arg

Drop stage1->stage2 action transfer; dismiss_sync self-determines no-op

Resolve PR number by head branch, not head SHA, to avoid a fast-iteration race

Disable the example checklist

Move review-checklists action to tools/review-checklists; add README

Post checklist findings as file-level comments to support binary files

Restrict compile_pip_requirements to linux

Validate checklist evidence in merge queue instead of assuming it

Transfer PR number from trigger to apply stage via artifact

Fix merge_group evidence validation never actually running

Switch checklist path matching from fnmatch to pathspec (gitignore-style)

Reduce visibility

Fix stale copyright-checker srcs entry after actions/ -> tools/ move

Add cancel-in-progress to review-checklists apply workflow concurrency

Document why review-checklists action installs deps with pip, not Bazel

Clarify Otterdog status-check syntax for the review-checklists status

Deduplicate OK-reply lookup across review-checklists scripts

Use Bzlmod module name instead of hardcoded '_main' for Rlocation

Document why get_approving_reviewers' discard call is not redundant

Name the commit-status description length magic number

Document why is_pr_in_merge_queue fails closed to False on error

Stay pending until every relevant checklist has been posted

Clarify why the merge-queue notice check is separate from merge_group handling

Avoid single/two-letter abbreviations for checklist variables

Improve remaining unclear variable names

Rename relevant_ids/affected_ids to posted_relevant_ids/posted_affected_ids

Rename relevant/affected checklist-list variables for clarity