Skip to content

Fix bundle size badge in README - #3600

Merged
RobbieTheWagner merged 1 commit into
mainfrom
fix-bundle-size-badge
Nov 11, 2026
Merged

RobbieTheWagner merged 1 commit into
mainfrom
fix-bundle-size-badge

Conversation

@RobbieTheWagner

@RobbieTheWagner RobbieTheWagner commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

The bundle size badge was broken: bundlephobia rate-limits badge services, so shields.io rendered "rate limited by upstream service" (and badgen's bundlephobia badge returns 502/429).

This switches to badgen's live bundlejs badge, which needs no maintenance, and links it to the bundlejs page for shepherd.js. It currently shows minzipped size: 11 KB (bundlejs measures slightly differently than bundlephobia's ~10.5 kB).

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the README badge to show the minzipped bundle size and link to BundleJS.

Bundlephobia rate-limits badge services, so the shields.io badge showed
"rate limited by upstream service". Switch to badgen's live bundlejs
badge.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
shepherd-docs Ready Ready Preview Nov 11, 2026 4:28pm UTC
shepherd-landing Ready Ready Preview Nov 11, 2026 4:28pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The README replaces the npm bundle-size badge with a minzipped-size badge that links to BundleJS.

Changes: README badge

Layer / File(s) Summary
Replace bundle-size badge
README.md
The badge now reports minzipped size and links to BundleJS.

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

Change: Bug fix

Possibly related PRs

Suggested reviewers: chuckcarpenter

Merge Risk: Minimal · up to ea77d

This updates the README bundle size badge to use BundleJS instead of the rate-limited Bundlephobia source. It does not touch runtime code and is safe to merge.

Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the README bundle size badge correction, which is the main change in the pull request.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2118ca4e-b48d-49b6-a75d-664ac7797106
Commits

Reviewing files that changed from the base of the PR and between 61aeb7f and ea77df8.

Files selected for processing: 1

  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

@qltysh

qltysh Bot commented Nov 11, 2026

Copy link
Copy Markdown

Qlty


Coverage Impact

This PR will not change total coverage.

Commit ea77df8 · 🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@RobbieTheWagner
RobbieTheWagner merged commit d63991c into main Nov 11, 2026
10 checks passed
@RobbieTheWagner
RobbieTheWagner deleted the fix-bundle-size-badge branch November 11, 2026 16:29

This branch was successfully deployed

2 active deployments
Preview – shepherd-landing — ea77df80 Deployed Nov 11, 2026 by vercel[bot]
Preview – shepherd-docs — ea77df80 Deployed Nov 11, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant