Skip to content
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

Replace gulp-clean-css with postcss-cssnano (Case 161784) #30

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

polarbirke
Copy link
Member

@polarbirke polarbirke commented Jul 26, 2023

This is a continuation of the problem-solving started in #28 and #29.

gulp-clean-css has probably become stale (with its author completely unresponsive), and while clean-css still is the most downloaded CSS minification package on npm, its author has put the project into "maintenance mode" and will no longer actively work on it.

cssnano has an active group of contributors and fits seamlessly into the existing configuration as a postcss plugin.

Downsides:

  • significantly slower than clean-css (between 0.5-1s depending on the project).

As it is logical to first purge unused CSS before performing any other transformations on the remaining code, the order of the postcss plugins is adjusted accordingly.

@polarbirke polarbirke changed the title Replace clean-css with cssnano Replace clean-css with cssnano (Case 161784) Jul 26, 2023
@polarbirke polarbirke changed the title Replace clean-css with cssnano (Case 161784) Replace gulp-clean-css with postcss-cssnano (Case 161784) Jul 26, 2023
@polarbirke polarbirke marked this pull request as draft July 27, 2023 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant