-
Notifications
You must be signed in to change notification settings - Fork 322
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
Add debug cmd switch #43
Comments
@XhmikosR quick question - would you like to see the same output as in
I'd go for the latter one. |
Yeah, I agree with the later one too. I don't think there is a gain by showing the time for each time of transformation. |
1st thought, is maybe mention the switch change in History.md. Secondly, I'm wondering, is "compression ratio" the right thing to show? How about showing the percentage of the original file the compressed file is? |
Good point with History.md. Maybe it shouldn't be called "ratio" but instead something like "compression efficiency" as it is calculated as:
|
I think that's the difference then. |
👍 |
Ok, that's it. Merging to master... |
@GoalSmashers: Do you think you could expose Example: var packCss = cleanCSS.process(inCss2, {
removeEmpty: true,
keepSpecialComments: 0,
debug: true
}); |
We will do once #138 is done. Current stateless design is pretty messed up. Please open a separate ticket for it. On 7 Sep 2013, at 11:42, XhmikosR notifications@github.com wrote:
|
..which would show the time spent and the compression ratio for the task.
The text was updated successfully, but these errors were encountered: