build(package): annotate sideEffects for CSS tree-shaking#1962
Conversation
Closes #1846. Co-authored-by: Cursor <cursoragent@cursor.com>
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThis PR adds package-level ChangesSide Effects & Tree-Shaking Documentation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed due to a network error. Comment |
CoverageThis report compares the PR with the base branch. "Δ" shows how the PR affects each metric.
Coverage improved or stayed the same. Great job! Run |
Summary
sideEffectsfor CSS/SCSS so bundlers can tree-shake unused JS while preserving stylesheet imports.Closes #1846.
Test plan
npm run check:exportspasses on commit.@radui/ui/themes/default.cssimports when components are tree-shaken.Made with Cursor
Summary by CodeRabbit
Documentation
Chores