Skip to content

✨ feat(reviewer): use GitHub severity badges - #34

Merged
SigureMo merged 3 commits into
mainfrom
feat/reviewer-severity-badges
Sep 26, 2026
Merged

SigureMo merged 3 commits into
mainfrom
feat/reviewer-severity-badges

Conversation

@SigureMo

@SigureMo SigureMo commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

What

  • Replace Shields priority labels with GitHub's High / Medium / Low severity badges in dashboard findings and inline-review guidance.
  • Standardize new findings on P1/P2/P3; P1 is the sole priority that can support REQUEST_CHANGES.
  • Keep P0 only as a legacy dashboard-data alias, rendered as High.

Validation

  • gh-slate render nyanpasu-review --config packages/nyanpasu-github-reviewer/src/nyanpasu_github_reviewer/templates/boards.toml --profile review --data packages/nyanpasu-github-reviewer/src/nyanpasu_github_reviewer/templates/review.example.json
  • uv run pytest packages/nyanpasu-github-reviewer/tests (55 passed)
  • git diff --check
This PR is co-authored with @copilot (gpt-6-luna xhigh)

@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@SigureMo SigureMo changed the title feat(reviewer): use GitHub severity badges ✨ feat(reviewer): use GitHub severity badges Sep 26, 2026
@SigureMo
SigureMo merged commit 4f36845 into main Sep 26, 2026
6 checks passed
@SigureMo
SigureMo deleted the feat/reviewer-severity-badges branch September 26, 2026 06:02
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