Fix the report option

[jpillora]

Fix the report option by using grunt.log instead of grunt.verbose

I upgraded from 0.2.x and there uglify had one grunt.log print:

File dist/1/xhook.min.js created: 13.23 kB → 5.82 kB → 2.52 kB (gzip)

It seems to only grunt.verbose print now. Maybe this was on purpose?

[UltCombo]

Related #254 #249 #229 #75

[jpillora]

I seeeee, quite the contentious issue. We don't need a 5th issue about this so I'll close. My vote is for at least one success print, or an option to enable this grunt.log, or for the report option to enable this grunt.log.

[commadelimited]

I know this issue is closed, but I'll chime in here. I'm writing a chapter about Grunt for a book I'm co-authoring. The docs have this to say about the report option:

report

Choices: 'min', 'gzip'
Default: 'min'

Either report only minification result or report minification and gzip results. This is useful to see exactly how well uglifyjs is performing but using 'gzip' will make the task take 5-10x longer to complete. Example output.

Nowhere in the docs does it indicate that -v is required to display the report. The docs should either be updated to reflect this requirement, or allow the report to display without verbose mode (my preference). I don't want ALL of the output, just the bits I'm explicitly requesting as per the documentation.

[STRML]

+1 for this, doesn't make sense to have the option if we can't see the output anyway.

[zachleat]

+1 for non-verbose report output. The default report should be empty, with non-empty values for report showing non-verbose output.

[leandono]

+1 for the @zachleat solution, the option needs 3 values: false (default), min or gzip.

[zachleat]

For future visitors, I believe the discussion for this issue is currently taking place at #254.