Skip to content

fix: remove unused braces dependencies#19797

Merged
RobinMalfait merged 2 commits intotailwindlabs:mainfrom
SidWuChou:fix/remove-unused-dependencies
Mar 15, 2026
Merged

fix: remove unused braces dependencies#19797
RobinMalfait merged 2 commits intotailwindlabs:mainfrom
SidWuChou:fix/remove-unused-dependencies

Conversation

@SidWuChou
Copy link
Copy Markdown
Contributor

@SidWuChou SidWuChou commented Mar 14, 2026

Removes unused braces and @types/braces from @tailwindcss/upgrade package.json as reported in issue #19794

fix: remove unused braces dependencies
e2b9138 
braces and @types/braces are not used anywhere in the codebase
@SidWuChou SidWuChou requested a review from a team as a code owner March 14, 2026 07:03
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: f196af0b-6903-43e0-9f89-517bfb5f3724

📥 Commits

Reviewing files that changed from the base of the PR and between a4be983 and e2b9138.

📒 Files selected for processing (1)
  • packages/@tailwindcss-upgrade/package.json
💤 Files with no reviewable changes (1)
  • packages/@tailwindcss-upgrade/package.json

Walkthrough

This pull request modifies the package configuration for the tailwindcss-upgrade package. The braces dependency has been removed from the dependencies section, and its corresponding type declarations have been removed from devDependencies. This represents a net removal of two lines from the package.json file with no modifications to exported APIs or functional behavior.

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: remove unused braces dependencies' directly and clearly describes the main change in the changeset, which removes unused braces and @types/braces dependencies.
Description check ✅ Passed The description is directly related to the changeset, explaining that it removes unused braces and @types/braces from package.json in response to issue #19794.

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

📝 Coding Plan
  • Generate coding plan for human review comments

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.

Tip

You can generate walkthrough in a markdown collapsible section to save space.

Enable the reviews.collapse_walkthrough setting to generate walkthrough in a markdown collapsible section.

RobinMalfait
RobinMalfait approved these changes Mar 15, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@RobinMalfait RobinMalfait left a comment

Choose a reason for hiding this comment

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

Thanks! We use the braces package in one of the other packages, but good catch here!

@RobinMalfait RobinMalfait enabled auto-merge (squash) March 15, 2026 21:06
@RobinMalfait RobinMalfait merged commit d24b112 into tailwindlabs:main Mar 15, 2026
9 checks passed
@RobinMalfait RobinMalfait mentioned this pull request Mar 16, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RobinMalfait RobinMalfait RobinMalfait approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SidWuChou @RobinMalfait