search

google-code-export / wro4j

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

Additional JSHint property: ignoreWarnings #689

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
It would be good to support the following as configuration property for the  
JSHint goal:

ignoreWarnings: JSHint issues 2 types of notifications: errors and warnings. 
See: https://github.com/jshint/jshint/blob/master/src/shared/messages.js

It would be nice to have the ability to ignore warnings for flexible/loose 
checking.

Thanks,
Simo

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

GoogleCodeExporter commented 9 years ago

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

GoogleCodeExporter commented 9 years ago 
Currently jshintProcessor doesn't handle warnings (errors only), meaning that 
they are ignored by default. What exactly is the benefit of adding this new 
property?

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

GoogleCodeExporter commented 9 years ago 
I thought of ignoreWarnings as a severity flag. But with the ability to 
customize jshint options this can be achieved indirectly.

Original comment by webinven...@gmail.com on 10 Mar 2013 at 10:46

GoogleCodeExporter commented 9 years ago 
In that case, can I close this issue? If you still find it useful, please 
update the description with acceptance criteria and more descriptions.

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

GoogleCodeExporter commented 9 years ago 
Yes, you can close it. Thanks!

Original comment by webinven...@gmail.com on 10 Mar 2013 at 11:43

GoogleCodeExporter commented 9 years ago

Original comment by alex.obj...@gmail.com on 21 Mar 2013 at 9:37