Skip to content

docs(readme): the badge advertised a daily ranking the repo has since beaten - #226

Merged
joyful-ii-V-I merged 1 commit into
mainfrom
chore/trendshift-weekly-badge
Sep 14, 2026
Merged

joyful-ii-V-I merged 1 commit into
mainfrom
chore/trendshift-weekly-badge

Conversation

@joyful-ii-V-I

Copy link
Copy Markdown
Collaborator

What

The Trendshift badge in the README pointed at the daily C++ feed. The repository took #1 C++ for the week, so the badge was understating its own result.

Two substitutions on one line:

  • repositories/217924/daily?language=C%2B%2Brepositories/217924/weekly?language=C%2B%2B
  • alt text Trendshift: C++ Repository of the Day badge for redhat-et/ripwire… Repository of the Week badge …

The alt text moves with the image so the accessible name states the same fact the image does.

What is deliberately unchanged

The repository id 217924, all three utm_ parameters, and the width/height. House markup style is kept as-is: descriptive alt text, no target= or rel=, and > rather than />.

Gates

None apply. I checked before writing the commit: nothing under test/ matches the badge image URL or the centred-paragraph wrapper, so this moves no pin and changes no gate. It is one line of README markup.

Split out of the 0.6.1 release PR on purpose — a weekly ranking is time-sensitive and the release has several lanes still landing, so this should not wait for them.

🤖 Generated with Claude Code

… beaten

The Trendshift badge pointed at the DAILY C++ feed. The repository took #1 C++
for the WEEK, so the badge was understating its own result: a reader saw the
narrower ranking while the wider one was the true one.

Two substitutions on one line, and nothing else. The image source moves from
`repositories/217924/daily` to `.../weekly`, and the alt text from "Repository
of the Day" to "Repository of the Week" so the accessible name states the same
fact the image does. The repository id, the three utm parameters and the
dimensions are untouched, as is the house markup style: descriptive alt text,
no `target=` or `rel=`, and `>` rather than `/>`.

No gate reads the badge (checked: nothing in test/ matches the image URL or the
centred-paragraph wrapper), so this carries no gate change and no pin to move.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@joyful-ii-V-I

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Sep 14, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 8bacfef8-bb1e-4c6c-baa6-1304e2064681

📥 Commits

Reviewing files that changed from the base of the PR and between c1915d2 and 6ba5802.

📒 Files selected for processing (1)
  • README.md

Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review.


📝 Summary

Summary by CodeRabbit

  • Documentation
    • Updated the README badge to display the weekly C++ repository ranking instead of the daily ranking.
    • Revised the badge description to reflect the weekly ranking.

Walkthrough

The README Trendshift badge now links to the weekly C++ badge image. Its alt text now identifies it as the repository-of-the-week badge.

Changes

Trendshift badge

Layer / File(s) Summary
Update badge reference
README.md
The badge image URL changes from daily to weekly, and the alt text changes from “C++ Repository of the Day badge” to “C++ Repository of the Week badge”.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Suggested reviewers: quaterniondrift

Merge Risk: ⚪ Minimal · up to 6ba58

The weekly badge URL and matching alt text are consistent, with no merge-blocking impact.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed 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…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the README badge change and explains that the daily ranking was outdated.
Description check ✅ Passed The description directly explains the switch from the daily to the weekly Trendshift feed and the matching alt-text update.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/trendshift-weekly-badge

Comment @coderabbitai help to get the list of available commands.

@coderabbitai

coderabbitai Bot commented Sep 14, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Already reviewed the last commit. Use @coderabbitai full review to rerun a review of the entire changeset.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@joyful-ii-V-I
joyful-ii-V-I merged commit 4922004 into main Sep 14, 2026
31 checks passed
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.

1 participant