Skip to content

fix(reporters): show retry count for unfinished checks in github reporter [RED-1243] [ship] - #1568

Merged
sorccu merged 1 commit into
mainfrom
simo/RED-1243-github-reporter-retries
Oct 25, 2026
Merged

sorccu merged 1 commit into
mainfrom
simo/RED-1243-github-reporter-retries

Conversation

@sorccu

@sorccu sorccu commented Oct 25, 2026

Copy link
Copy Markdown
Member

Linear: RED-1243

When --max-failures stops a run early, the github reporter now shows how many times an unfinished check had already retried, e.g. ⏹️ Not finished (1 retry). Finished rows (✅ Pass (2 retries)) and the list/ci (#1566) and json reporters already show this.

The retry suffix is now built once per row and shared by both branches. A new unit test fails without the fix.

This finishes #1565. Merging #1566 auto-closed that issue, but #1566 only fixed the list reporter.

🤖 Generated with Claude Code

…rter [RED-1243]

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sorccu sorccu changed the title fix(reporters): show retry count for unfinished checks in github reporter [RED-1243] fix(reporters): show retry count for unfinished checks in github reporter [RED-1243] [ship] Oct 25, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved: ship/show PR from a same-repo branch.

@sorccu
sorccu merged commit cf77b62 into main Oct 25, 2026
27 of 29 checks passed
@sorccu
sorccu deleted the simo/RED-1243-github-reporter-retries branch October 25, 2026 10:49
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