Skip to content

docs: change <Tabs /> to <CodeGroup /> for package managers#4291

Merged
chronark merged 1 commit intounkeyed:mainfrom
pqoqubbw:main
Nov 10, 2025
Merged

docs: change <Tabs /> to <CodeGroup /> for package managers#4291
chronark merged 1 commit intounkeyed:mainfrom
pqoqubbw:main

Conversation

@pqoqubbw
Copy link
Contributor

@pqoqubbw pqoqubbw commented Nov 9, 2025

What does this PR do?

Old New
CleanShot 2025-11-09 at 8  22 39@2x CleanShot 2025-11-09 at 8  23 18@2x

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • Enhancement (small improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How should this be tested?

cd apps/docs
npx mint dev

Checklist

Required

  • Filled out the "How to test" section in this PR
  • Read Contributing Guide
  • Self-reviewed my own code
  • Commented on my code in hard-to-understand areas
  • Ran pnpm build
  • Ran pnpm fmt
  • Ran make fmt on /go directory
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR
  • Updated the Unkey Docs if changes were necessary

@changeset-bot
Copy link

changeset-bot bot commented Nov 9, 2025

⚠️ No Changeset found

Latest commit: 41bf127

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Nov 9, 2025

@pqoqubbw is attempting to deploy a commit to the Unkey Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Nov 9, 2025

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 9, 2025

📝 Walkthrough

Walkthrough

This PR restructures installation command displays across seven documentation files by replacing Tabs-based UI components with CodeGroup components. The package manager commands (npm, pnpm, yarn, bun) remain functionally identical; only their presentation markup changes from individual tab blocks to grouped code blocks with language tags.

Changes

Cohort / File(s) Summary
Library Documentation
apps/docs/libraries/nuxt/overview.mdx, apps/docs/libraries/ts/cache/overview.mdx, apps/docs/libraries/ts/ratelimit/ratelimit.mdx
Converted installation instructions from Tabs-based layout to CodeGroup with npm/pnpm/yarn/bun language-tagged code blocks, preserving all commands and order
Quickstart API Guides
apps/docs/quickstart/apis/hono.mdx, apps/docs/quickstart/apis/nextjs.mdx
Replaced per-step tabbed layouts (Create Application, Install, Running it) with unified CodeGroup blocks for each package manager; all commands unchanged
Quickstart Ratelimiting Guides
apps/docs/quickstart/ratelimiting/hono.mdx, apps/docs/quickstart/ratelimiting/nextjs.mdx
Converted multi-tab installation and execution steps to CodeGroup-wrapped code blocks across Create Application, Install, and Running it sections; command content preserved

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Homogeneous, repetitive changes across all files following a single pattern (Tabs → CodeGroup conversion)
  • No logic, behavioral, or functional changes—purely markup restructuring
  • Visual verification needed to confirm commands are accurately preserved in new format

Possibly related PRs

Suggested labels

Documentation, Needs Approval

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title directly summarizes the main change: replacing Tabs components with CodeGroup components for package manager installation instructions across documentation files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description includes a visual comparison showing the before/after changes, specifies the type of change as 'Enhancement', provides clear testing instructions, and completes most required checklist items.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 9, 2025

Thank you for following the naming conventions for pull request titles! 🙏

@vercel
Copy link

vercel bot commented Nov 9, 2025

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

Project Deployment Preview Comments Updated (UTC)
dashboard Ready Ready Preview Comment Nov 9, 2025 7:28pm
engineering Ready Ready Preview Comment Nov 9, 2025 7:28pm

Copy link
Collaborator

@chronark chronark left a comment

Choose a reason for hiding this comment

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

tysm

@chronark chronark enabled auto-merge November 10, 2025 08:38
@chronark chronark added this pull request to the merge queue Nov 10, 2025
@graphite-app
Copy link

graphite-app bot commented Nov 10, 2025

Video gif. A toddler grins wide, raising their hand and giving an exuberant thumbs up at us. Their nose krinkles under their big smile.  (Added via Giphy)

@graphite-app
Copy link

graphite-app bot commented Nov 10, 2025

Graphite Automations

"Post a GIF when PR approved" took an action on this PR • (11/10/25)

1 gif was posted to this PR based on Andreas Thomas's automation.

Merged via the queue into unkeyed:main with commit b697f78 Nov 10, 2025
18 of 21 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.

4 participants