Skip to content

Fix download statistics #706

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

indygreg
Copy link
Collaborator

just download-stats* are failing with an HTTP 504 Gateway Timeout on GitHub's servers.

We can work around this by reducing the requested number of releases per page.

As part of this, we adopt a 3rd party package for querying the GitHub API because I was too lazy to implement pagination manually. Also, since pagination is enabled, the counts reported by these commands are now accurate and don't implicitly truncate at the most recent 10 releases.

@indygreg indygreg force-pushed the indygreg/fix-download-stats branch 2 times, most recently from d1140d1 to 6c8534d Compare July 21, 2025 03:47
`just download-stats*` are failing with an HTTP 504 Gateway Timeout
on GitHub's servers.

We can work around this by reducing the requested number of releases
per page.

As part of this, we adopt a 3rd party package for querying the
GitHub API because I was too lazy to implement pagination manually.
Also, since pagination is enabled, the counts reported by these
commands are now accurate and don't implicitly truncate at the most
recent 10 releases.
@indygreg indygreg force-pushed the indygreg/fix-download-stats branch from 6c8534d to e8ef3a8 Compare July 22, 2025 01:03
@zanieb zanieb added the ci:skip label Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants