Skip to content

docs: add Trendshift daily and weekly badges - #56

Merged
qiz029 merged 3 commits into
mainfrom
codex/readme-trendshift-badge
Aug 5, 2026
Merged

docs: add Trendshift daily and weekly badges#56
qiz029 merged 3 commits into
mainfrom
codex/readme-trendshift-badge

Conversation

@qiz029

@qiz029 qiz029 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Added the Trendshift daily Go repository badge to the README badge section.
  • Added the matching weekly Trendshift badge alongside it.

Why

This makes both the repository's daily and weekly Trendshift rankings visible alongside the existing CI, release, Go, and platform badges.

Impact

Documentation-only change; no runtime behavior is affected.

Validation

  • git diff --check

Copilot AI lite review requested due to automatic review settings July 26, 2026 18:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a Trendshift daily ranking badge to the README’s existing badge block so the repository’s Trendshift visibility appears alongside CI/release/platform status.

Changes:

  • Added a Trendshift daily Go repository badge link/image to the README badge section.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Copilot AI review requested due to automatic review settings August 5, 2026 20:43
@qiz029 qiz029 changed the title docs: add Trendshift badge docs: add Trendshift daily and weekly badges Aug 5, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

README.md:12

  • These badges are added as raw HTML while the rest of the badge block uses Markdown image links. This makes the README harder to maintain (very long lines) and the current alt text is URL-encoded (%2F), which is less accessible. Prefer consistent Markdown badge syntax with human-readable alt text.
<a href="https://trendshift.io/repositories/88474?utm_source=trendshift-badge&amp;utm_medium=badge&amp;utm_campaign=badge-trendshift-88474" target="_blank" rel="noopener noreferrer"><img src="https://trendshift.io/api/badge/trendshift/repositories/88474/daily?language=Go" alt="pax-beehive%2Fpaxm | Trendshift" width="250" height="55"/></a>
<a href="https://trendshift.io/repositories/88474?utm_source=trendshift-badge&amp;utm_medium=badge&amp;utm_campaign=badge-trendshift-88474" target="_blank" rel="noopener noreferrer"><img src="https://trendshift.io/api/badge/trendshift/repositories/88474/weekly?language=Go" alt="pax-beehive%2Fpaxm | Trendshift" width="250" height="55"/></a>

Copilot AI review requested due to automatic review settings August 5, 2026 20:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

README.md:12

  • The new badge alt-text includes the full repo name and a |, which is inconsistent with the other badges in this section (e.g., "CI", "Release", "Go", "Platforms") and can render awkwardly in some Markdown contexts. Consider simplifying the labels to just "Trendshift Daily" / "Trendshift Weekly" for consistency.
[![pax-beehive/paxm | Trendshift Daily](https://trendshift.io/api/badge/trendshift/repositories/88474/daily?language=Go)](https://trendshift.io/repositories/88474)
[![pax-beehive/paxm | Trendshift Weekly](https://trendshift.io/api/badge/trendshift/repositories/88474/weekly?language=Go)](https://trendshift.io/repositories/88474)

@qiz029
qiz029 merged commit 35de022 into main Aug 5, 2026
5 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.

2 participants