grunt-contrib-jshint icon indicating copy to clipboard operation
grunt-contrib-jshint copied to clipboard

Upstream console logger to jshint

Open jzaefferer opened this issue 11 years ago • 2 comments

One big strength of this plugin over using jshint directly is the console logger. Compare this:

sublimedemo.js: line 4, col 6, 'x' is defined but never used.

with

Linting sublimedemo.js ...ERROR
[L4:C6] W098: 'x' is defined but never used.
var x =1;

The above lacks the colored output this plugin actually provides, which also helps with the readability.

I don't yet know if there's any interest on the end of jshint, but I think we should at least try to upstream this logging implementation to make it available to all users of jshint, not just users of this plugin.

Looks to me like the relevant code is mostly this part in tasks/lib/jshint.js

jzaefferer avatar Mar 24 '14 17:03 jzaefferer

Hey @jzaefferer is this issue for us or for JSHint?

vladikoff avatar Apr 08 '14 21:04 vladikoff

@vladikoff well, eventually. It seems like the best approach would be for one the contributors on this project to put together a pull request for JSHint, since that should avoid any copyright issues. That way we can keep track of that PR and update this plugin when the JSHint output matches.

jzaefferer avatar May 19 '14 14:05 jzaefferer