guybedford / require-css

A RequireJS CSS loader plugin to allow CSS requires and optimization
MIT License
982 stars 365 forks source link

'Support both csso v1 and v2' - Improved code in handling csso optimization function (justDoIt vs. minify) #226

Closed alundiak closed 7 years ago

alundiak commented 7 years ago

csso releases review:

Tested with:

Compatibility Note: I realize, this change may be overthinking/overcomplex, but so far the fact is - there are end consumers of require-css who will upgrade to new require-css version, but remain old csso version --globalor as in my enterprise case (locally in project). They will have errors like I mentioned in this PR and in #203 and #219.

A bit of automation :) in this boring life

1) In scope of this PR, I added compatibility-test.sh. 2) To use this script, you should change optimizeCss in example/build.jsfile from "none"to

cc/ @guybedford @prantlf