Skip to content
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

feat: added star history chart #2527

Closed
wants to merge 2 commits into from

Conversation

github-star-history
Copy link

Star History chart added, hope you like it! :)

Checks

  • I am happy for maintainers to push small adjustments to this PR, to speed up the review cycle
  • I have checked that there are no existing pull requests for the same thing

@github-star-history github-star-history marked this pull request as draft January 6, 2025 03:17
@github-star-history github-star-history marked this pull request as ready for review January 6, 2025 03:17
@github-star-history github-star-history changed the title Update README.md Added star history chart Jan 6, 2025
@github-star-history github-star-history changed the title Added star history chart feat: added star history chart Jan 6, 2025
@ellie
Copy link
Member

ellie commented Jan 7, 2025

Looks like the API is down - I'd be open to this in theory, but it seems like this might be automated and unreliable

@ellie ellie closed this Jan 7, 2025
@github-star-history
Copy link
Author

github-star-history commented Jan 7, 2025

@ellie The API has maintained 100% uptime since December 2023. For repositories with a large number of stars, fetching all the data can take some time. To optimize performance, we cache images for 24 hours. When the cache expires, the first request triggers a cache refresh, which takes approximately 40 seconds for this repo. As a result, the first request after the 24-hour period may fail with a timeout error (since github add timeout restrictions and also heroku does). However, this triggers the update process, ensuring that all subsequent requests within the next 24 hours are served in just 2-3 milliseconds.

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.

2 participants