fix: restore broken star history chart - #1434
Conversation
The star history chart in the README is currently broken due to GitHub stargazer API restrictions. Switch the chart and its link to star-history.dera.page, an alternative that requires no API token.
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan includes up to 8 reviews per rolling hour; 7 remain after this review. 📝 WalkthroughWalkthrough两个 README 文件中的 Star History 图片地址和跳转链接已从原服务更新为 ChangesStar History 文档链接
Estimated code review effort: 1 (简单) | ~2 minutes Merge Risk: ⚪ Minimal · up to This is a narrowly scoped README-only change to restore the star history chart, with no actionable merge-blocking risk remaining beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Code Review Summary
This PR updates the Star History chart URLs in both README files to use an alternative provider (star-history.dera.page) due to GitHub stargazer API restrictions affecting the original service. The changes are straightforward and limited to documentation files.
PR Size: XS
- Lines changed: 4 (2 additions, 2 deletions)
- Files changed: 2 (README.md, README.en.md)
Issues Found
| Category | Critical | High | Medium | Low |
|---|---|---|---|---|
| Logic/Bugs | 0 | 0 | 0 | 0 |
| Security | 0 | 0 | 0 | 0 |
| Error Handling | 0 | 0 | 0 | 0 |
| Types | 0 | 0 | 0 | 0 |
| Comments/Docs | 0 | 0 | 0 | 0 |
| Tests | 0 | 0 | 0 | 0 |
| Simplification | 0 | 0 | 0 | 0 |
Review Coverage
- Logic and correctness - Clean
- Security (OWASP Top 10) - Clean
- Error handling - Clean
- Type safety - Clean
- Documentation accuracy - Clean
- Test coverage - N/A (documentation only)
- Code clarity - Good
Automated review by Claude AI
Summary
Fixes the broken star history chart in README files by switching from
api.star-history.comtostar-history.dera.page.Problem
The current star history chart provider (
api.star-history.com) is affected by GitHub stargazer API restrictions and no longer displays the chart correctly.Solution
Switch to
star-history.dera.page, an alternative provider that works without requiring an API token.Changes
README.md: Update star history chart URLsREADME.en.md: Update star history chart URLsBefore:
After:
Verification
The chart should now display properly at both URLs:
Note: This PR targets
mainbut per CLAUDE.md, PRs should typically targetdevbranch.Description enhanced by Claude AI