Skip to content

fix(packages/ui): hide branch tag on default branches#72

Merged
zrosenbauer merged 2 commits intomainfrom
fix/ui-branch-tag-hide-master
Mar 25, 2026
Merged

fix(packages/ui): hide branch tag on default branches#72
zrosenbauer merged 2 commits intomainfrom
fix/ui-branch-tag-hide-master

Conversation

@zrosenbauer
Copy link
Copy Markdown
Member

Summary

  • Hide the branch tag badge when the current branch is master (in addition to main)

Changes

  • branch-tag.tsx: added master to the default branch check

Testing

  • pnpm typecheck passes (packages/ui)

The branch tag badge was only hidden for `main`. Repos using `master`
as their default branch would still show the tag in production.

Co-Authored-By: Claude <noreply@anthropic.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 25, 2026

🦋 Changeset detected

Latest commit: ae707c6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@zpress/ui Patch
@zpress/cli Patch
@zpress/kit Patch

Not sure what this means? Click here to learn what changesets are.

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

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 25, 2026

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

Project Deployment Actions Updated (UTC)
oss-zpress Ready Ready Preview, Comment Mar 25, 2026 3:34am

Request Review

Co-Authored-By: Claude <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: f186eff9-3ca2-4808-b391-67bd284473a7

📥 Commits

Reviewing files that changed from the base of the PR and between 1e966e1 and ae707c6.

📒 Files selected for processing (2)
  • .changeset/hide-branch-tag-master.md
  • packages/ui/src/theme/components/nav/branch-tag.tsx

📝 Walkthrough

Walkthrough

A changeset entry is added to mark a patch release for the UI package. The corresponding component modification extends the branch tag visibility logic to hide the badge when the repository's default branch is either main or master. Previously, the component only hid the badge for the main branch. The BranchTag() function now checks for both common default branch names before rendering, with all other rendering behavior remaining unchanged.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: hiding the branch tag badge on default branches (main and master).
Description check ✅ Passed The description is directly related to the changeset, clearly explaining that master was added to the default branch check with testing confirmation.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

@zrosenbauer zrosenbauer merged commit 0113fb1 into main Mar 25, 2026
5 checks passed
@zrosenbauer zrosenbauer deleted the fix/ui-branch-tag-hide-master branch March 25, 2026 03:44
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