-
-
Notifications
You must be signed in to change notification settings - Fork 486
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(css_formatter): don't wrap selector identation after css comments #4614
Conversation
8a35131
to
c2e3cda
Compare
CodSpeed Performance ReportMerging #4614 will not alter performanceComparing Summary
|
b171329
to
ea1870b
Compare
@@ -1,7 +1,6 @@ | |||
--- | |||
source: crates/biome_formatter_test/src/snapshot_builder.rs | |||
info: css/issue_3229.css | |||
snapshot_kind: text |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems your local version of cargo-insta
is outdated. You can update it using just update-tools
, or just cargo install cargo-insta
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the reminder, I have updated it
aae5943
to
c50e0de
Compare
c50e0de
to
b566925
Compare
b566925
to
14ec4b2
Compare
|
||
.aRule { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary
closes: #4575
I add special judge for the case like:
if there was a selector, it contains comments before, it will don't be wrapped by the ident(So i add judge in this case to remove the ident for the selector).
Test Plan
I add test case.