Skip to content

docs: fix consistency of byte units (kb to kB) - #77

Open
KoHaRxnP wants to merge 3 commits into
dashersw:mainfrom
KoHaRxnP:main
Open

docs: fix consistency of byte units (kb to kB)#77
KoHaRxnP wants to merge 3 commits into
dashersw:mainfrom
KoHaRxnP:main

Conversation

@KoHaRxnP

@KoHaRxnP KoHaRxnP commented Jul 2, 2026

Copy link
Copy Markdown

I noticed that while the tables use uppercase B for bytes (e.g., 214 B), the text sections and the comparison table use lowercase kb.

I've updated the lowercase kb instances to kB to keep the unit notation accurate and consistent throughout the README.


P.S. I love Gea's philosophy so much that I've just built a data-fetching library for it called gea-query! I also opened an issue (#76 ) to propose adding it to the ecosystem docs. If you have some time, I'd be honored if you could check it out as well!

Summary by CodeRabbit

Summary by CodeRabbit

  • Documentation
    • Corrected size-unit labels in benchmark and comparison text, changing “kb” to “kB” for Brotli JavaScript figures.
    • Standardized wording across hello-world, interactive todo, baseline, and bundle-size examples.

@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 88f00a57-6aa1-498d-970f-71f86b4a8023

📥 Commits

Reviewing files that changed from the base of the PR and between 45e5d37 and c01d89f.

📒 Files selected for processing (1)
  • README.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • README.md

📝 Walkthrough

Walkthrough

This PR corrects brotli JavaScript size units in four README.md sections, changing “kb” to “kB”.

Changes

README size-unit corrections

Layer / File(s) Summary
Unit label corrections across README sections
README.md
Corrects brotli JavaScript size units from “kb” to “kB” in the hello-world, interactive todo, near-zero baseline, and bundle-size comparison sections.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
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 change from lowercase kb to uppercase kB for consistent byte units.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

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