Skip to content

ci: severity-classified sticky review comments#21

Merged
fonkamloic merged 1 commit into
mainfrom
ci/review-severity
Jun 12, 2026
Merged

ci: severity-classified sticky review comments#21
fonkamloic merged 1 commit into
mainfrom
ci/review-severity

Conversation

@fonkamloic

Copy link
Copy Markdown
Contributor

Updates the automated review workflow:

  • Findings are classified Critical / Medium / Low; per our review policy, only Critical blocks merging and Medium/Low may be tabled.
  • The reviewer reads existing PR comments and threads before raising findings, and must cite line numbers when claiming control-flow gaps — prevents re-raising refuted findings across re-reviews.
  • use_sticky_comment: true: re-reviews update one comment instead of stacking new ones.

Note: the review action skips on this PR itself (workflow file differs from main until merged) — that's its anti-tamper validation working as intended.

@fonkamloic fonkamloic merged commit ba2bc51 into main Jun 12, 2026
1 check failed
@fonkamloic fonkamloic deleted the ci/review-severity branch June 12, 2026 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant