Skip to content

fix(release-stats): omit zero-download releases from the by-release chart - #317

Merged
zheli merged 1 commit into
mainfrom
zzz/61ea4d9e
Aug 18, 2026
Merged

zheli merged 1 commit into
mainfrom
zzz/61ea4d9e

Conversation

@zheli

@zheli zheli commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

The README "Total downloads, by release" chart included tags with zero downloads, which flattened the series. Filter those releases out of the generated SVG so only versions with actual downloads appear.

Changes

  • In scripts/release-stats.sh, keep only releases with total > 0 when building the by-release line chart.

Test plan

  • make test
  • make lint
  • scripts/e2e/run-all.sh
  • bash -n scripts/release-stats.sh
  • After merge, run the Release stats workflow (or bash scripts/release-stats.sh) so release-downloads-by-version.svg is regenerated on release-stats-data

Checklist

  • Tests pass locally
  • No coverage regression on touched code
  • Docs updated (and website/ synced if any mirrored docs/*.md changed)
  • CLI ↔ Web UI parity maintained (or follow-up issue + TODO(#issue) comment)

…hart

The README chart was plotting every tag, including unused releases that flatten the series. Filter those out so the chart only shows versions with actual downloads.
@zheli
zheli marked this pull request as ready for review August 18, 2026 12:22
@zheli
zheli merged commit c7a83bf into main Aug 18, 2026
@zheli
zheli deleted the zzz/61ea4d9e branch August 18, 2026 12:22
zheli added a commit that referenced this pull request Aug 19, 2026
…hart (#317)

The README chart was plotting every tag, including unused releases that flatten the series. Filter those out so the chart only shows versions with actual downloads.
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