docs(readme): fix broken star history chart - #199
Conversation
The embedded star history chart is currently broken: GitHub restrictions on the stargazer API have taken the upstream star-history.com chart down. Point the chart and its wrapping link at the maintained star-history.dera.page alternative, which uses a different data source that requires no API token.
|
🎉 Thanks for your first pull request to floci-az! Your CI checks need a maintainer to approve them before they run. That is GitHub's standard gate on first-time contributors, not a problem with your PR — so if the checks look like they are doing nothing, that is why. Once a maintainer approves, CI and the compatibility suite start automatically. Nothing is needed from you in the meantime. While you wait, a couple of things that make review faster:
Come join us in Slack — it is the fastest way to reach maintainers if you get stuck, or want feedback on an approach before investing more time in it. |
|
| Filename | Overview |
|---|---|
| README.md | Updates the star-history chart and link using consistent repository identifiers and valid replacement-service URL formats; no issues found. |
Reviews (1): Last reviewed commit: "docs(readme): fix broken star history ch..." | Re-trigger Greptile
|
Thank you for taking the time to send a first PR here, and you are right, the chart does render broken on the README. What seems to be going on: the endpoint itself answers fine when fetched directly (api.star-history.com/chart?repos=floci-io/floci returns a valid SVG), but GitHub serves README images through its camo proxy, and star-history's rate limiting throttles those shared proxy IPs, so the badge fails on the page even while the service is up. Your replacement does render reliably, but it is a different provider rather than the star-history project, and pointing every visitor's image request at an unofficial host is a trust call we would rather not make for a cosmetic chart. (blocking) Given that, our preference is to drop the section for now and maybe revisit a chart later. Would you repurpose this PR to remove the Star History section entirely instead? That is the whole block under the ## Star History heading in README.md, heading included, and a retitle to something like docs: remove broken star history chart. Same problem you found, simpler durable fix, and the credit stays yours. |
The star history chart in the README is currently broken because the upstream service can no longer fetch stargazer data due to GitHub API restrictions.
This change moves the chart to the maintained star-history.dera.page alternative, which uses a different data source that requires no API token, and updates the wrapping link to the new domain as well. No other references are affected.