Skip to content

fix(release): stop CI progress spam from spinner ticks - #181

Merged
pfeerick merged 1 commit into
mainfrom
fix/release-ci-progress-spam
Aug 30, 2026
Merged

pfeerick merged 1 commit into
mainfrom
fix/release-ci-progress-spam

Conversation

@pfeerick

Copy link
Copy Markdown
Member

Summary

  • Rich's Progress bar auto-refreshes ~10x/sec by default. On a real terminal that just redraws the same line in place, but GitHub Actions' live log tail can't overwrite a line — so every refresh became its own permanent log line, producing several near-duplicate lines per file while ffmpeg/ffmpeg-normalize processed it (e.g. 4 lines for the same file, differing only by spinner glyph), instead of one line per file actually finished.
  • Disable auto_refresh when CI is set, and call progress.refresh() manually only right after progress.advance(), so CI logs get one line per unit of real progress instead of one per spinner tick. Local/interactive runs are unaffected (still auto-refresh live as before).

Test plan

  • Simulated CI=true against an isolated fixture and confirmed the archive-creation step now emits one progress frame per zip created instead of a stream of spinner-tick duplicates.
  • CI run of release.py shows one log line per file/archive processed rather than several per item.

Rich's Progress bar auto-refreshes ~10x/sec by default. On a real
terminal that just redraws the same line in place, but GitHub Actions'
live log tail can't overwrite a line, so every refresh became its own
permanent log line -- several near-duplicate lines per file while
ffmpeg/ffmpeg-normalize processed it, not one per file.

Disable auto_refresh on CI and refresh manually only when
progress.advance() actually happens, so CI logs get one line per item
of real progress instead of one per spinner tick.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@pfeerick
pfeerick force-pushed the fix/release-ci-progress-spam branch from e5dda7b to 9ad1d71 Compare August 30, 2026 03:41
@pfeerick
pfeerick merged commit 4e34b6c into main Aug 30, 2026
2 checks passed
@pfeerick
pfeerick deleted the fix/release-ci-progress-spam branch August 30, 2026 03:42
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