Skip to content

Aggregate PR checks status in the review overlay#21

Merged
SDSLeon merged 2 commits into
masterfrom
lightcode/brave-jaguar
May 24, 2026
Merged

Aggregate PR checks status in the review overlay#21
SDSLeon merged 2 commits into
masterfrom
lightcode/brave-jaguar

Conversation

@SDSLeon

@SDSLeon SDSLeon commented May 24, 2026

Copy link
Copy Markdown
Owner
  • Feature + refactor: unify PR-level and cached PR-detail checks state so the review overlay reflects the latest checks result instead of stale partial data.
  • Refresh pending PR data when checks are still unresolved, including cached PR details, to keep the overlay in sync with the remote PR.
  • Surface checks status in the tab pill so the Checks icon shows success, warning, or failure at a glance.
  • Add coverage for the refresh path and the tab icon color states.

SDSLeon added 2 commits May 24, 2026 16:45
- Add helpers in `prStatus.ts` to aggregate and map checks to status tones.
- Introduce `usePrCombinedChecksStatus` to compute a unified checks status.
- Update `usePendingPrRefresh` to trigger auto-refreshes when checks are pending.
- Pass the combined status to `PrTabs` and `PrTabsPill` to color the Checks icon.
- Add test coverage for the pending refresh hook and the tab pill icon styling.
@vercel

vercel Bot commented May 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lightcode-landing Ready Ready Preview, Comment May 24, 2026 11:53pm

@SDSLeon

SDSLeon commented May 24, 2026

Copy link
Copy Markdown
Owner Author

hi github!

@SDSLeon SDSLeon merged commit 0f8ab99 into master May 24, 2026
6 checks passed
@SDSLeon SDSLeon deleted the lightcode/brave-jaguar branch May 24, 2026 23:57
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