search

jahaziel17 / wro4j

Automatically exported from code.google.com/p/wro4j
0 stars 0 forks source link

JSHint execution summary #690

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
when JSHint completes its task, it would be nice to output a summary of 
errors/warnings and # of files being affected. e.g.

[INFO] ---------------------------------------------------
[INFO] JSHINT found 120 errors, 450 warnings in 250 files.
[INFO] ---------------------------------------------------

Thanks,
Simo

Original issue reported on code.google.com by simo.mou...@gmail.com on 8 Mar 2013 at 6:28

GoogleCodeExporter commented 8 years ago

Original comment by alex.obj...@gmail.com on 9 Mar 2013 at 11:19

GoogleCodeExporter commented 8 years ago 
Fixed in branch 1.6.x.

The only problem is that there is no number of warnings available (processor 
doesn't expose this information).

Original comment by alex.obj...@gmail.com on 9 Mar 2013 at 11:57

GoogleCodeExporter commented 8 years ago 
Great, no warnings is fine for now. I'm guessing this cripples the 
"ignoreWarnings" feature as well. 

Also, do you think we should rather indicate the file count for the files with 
errors and not the total processed files. I thought that "Found 14 errors in 3 
files" would imply that all 3 files contain errors and precisely 14 of them.

Original comment by simo.mou...@gmail.com on 10 Mar 2013 at 1:19

GoogleCodeExporter commented 8 years ago 
Another recommendation is when no errors are found, showing "JSHINT found 0 
errors in 0 files" is not as good as:

JSHINT found no errors.

Original comment by simo.mou...@gmail.com on 10 Mar 2013 at 7:05

GoogleCodeExporter commented 8 years ago 
@simo applied changes according to your suggestions.

Original comment by alex.obj...@gmail.com on 10 Mar 2013 at 7:45