refactor(css): replace typed bogus nodes with CssBogus#9436
refactor(css): replace typed bogus nodes with CssBogus#9436denbezrukov wants to merge 3 commits intomainfrom
Conversation
|
Parser conformance results onjs/262
jsx/babel
markdown/commonmark
symbols/microsoft
ts/babel
ts/microsoft
|
Merging this PR will improve performance by 12.75%
Performance Changes
Comparing Footnotes
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
WalkthroughThis PR collapses many specialised "bogus" CSS variants into a single Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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. Comment |
…rregularWhitespace lint
Summary
Replaces CSS typed bogus nodes with the shared
CssBogusnode to reduce the number of CSS syntax kinds.The CSS grammar had grown enough SCSS and at-rule support that it exceeded the fixed
SyntaxKindSetcapacity inbiome_rowan, which forced an undesirable global bitset-size bump.Test Plan
cargo test -p biome_css_parser
cargo test -p biome_css_formatter