docs: update README download badges - #229
Conversation
Use the desktop release version and the site-matched download count. Task: README download and desktop release badges.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Included review availability: 6 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 10 reviews per hour. 📝 WalkthroughWalkthroughThe workflow now generates desktop and combined download badges, including npm counts, and commits the new total endpoint. The README updates its release and download badge links and removes the license badge. ChangesDownload badge refresh
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This change updates README badges and the badge-refresh workflow without affecting shipped application behavior, dependencies, or package data. No actionable merge-blocking risk remains after normal checks and review. Sequence Diagram(s)sequenceDiagram
participant GitHub as GitHub Actions
participant Releases as Desktop releases
participant Npm as npm downloads API
participant Repository as Repository files
GitHub->>Releases: Read desktop release assets
GitHub->>Npm: Retrieve npm download count
GitHub->>GitHub: Select the larger count
GitHub->>Repository: Write and commit badge endpoints
Repository->>Repository: Update README badge references
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description covers the related-issue exception, problem, changes, risks, verification, rollback, review points, and checklist. It is complete for this maintainer-requested documentation and workflow change. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.) Comment |
commit: |
Related Issue
No issue. This is a maintainer-requested README and badge workflow update.
Problem
The README shows the npm package version as the release and counts only desktop downloads. This
does not match the desktop-focused README or the combined counter on code.pythinker.com.
What changed
PyModel/pythinker-desktop-releases.downloads-total.jsonas the larger of desktop downloads and npm last-year downloads,matching the site formula.
main, while retaining the daily refresh.Risk
badgesbranch.Verification
550, npm2857, published downloads2857.license badge.
v0.3.9.passed.
Rollback and review
Checklist
Summary by CodeRabbit
New Features
Documentation