-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Use single line per selector & don't strip license comments #6919
Conversation
What is the size difference? |
Codecov Report
@@ Coverage Diff @@
## master #6919 +/- ##
=========================================
Coverage ? 41.41%
=========================================
Files ? 441
Lines ? 59736
Branches ? 0
=========================================
Hits ? 24738
Misses ? 31760
Partials ? 3238 Continue to review full report at Codecov.
|
About few bytes. could be saved by removing some insane nesting like Normally a linter is used to avoid stuff like this and get an more consistent less/sass file. |
Why?
Every PR touching css needs a rebase after a single change in an merged PR. Single line will only conflict if the same line/area is modified. Less Builds, less conflicts
Also i added -s0 option. No comments with
?*!
will be removed (used for licenses).I also would like the source map option, but isn't in code like all other options.
Doku
https://github.com/jakubpawlowicz/clean-css/tree/v3.0.1#how-to-use-clean-css-cli