-
Notifications
You must be signed in to change notification settings - Fork 175
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
Remove compress option? #27
Comments
I'd be happy to remove the compress option from this task too. Right now I'm firefighting some multi-file handling issues with this task and |
Thanks! I looked at the source before, but was confused with a few things:
cheers |
Hey, I made a pull request: #28 Keep up with good work! |
@tomByrer: Ouch, you're right. I was being sloppy. Fixed it, thanks! |
Welcome. I did the same thing; submitting after changing only 1 file. Then I remembered there was more than one file in the repo :) |
UnCSS removed the compress option (it is all @sindresorhus fault ;). I do see you are using cssmin. It is a good compressor, though I believe many devs have their own favorite minifier.
I can go either way, though I lean towards no minifer. But I wonder if it is more accurate/faster to lightly (non-aggressive) minify CSS first then reduces... hmmm...
I'm just opening this issue up for discussion.
The text was updated successfully, but these errors were encountered: