fnd / jslint-reporter Goto Github PK

see https://github.com/FND/jslint-reporter/compare/jshint - but needs an option to switch back and forth between the two

It would be simpler to use jslint-reporter the first time with the automatic download of jslint.js.

hello.

this command warn me this error message :

TypeError: Cannot read property 'edition' of undefined
at main (/home/christophe/dotfiles/emacs/.emacs.d/jslint-reporter/wrapper.js:69:42)
at /home/christophe/dotfiles/emacs/.emacs.d/jslint-reporter/wrapper.js:51:4
at IncomingMessage. (/home/christophe/dotfiles/emacs/.emacs.d/jslint-reporter/wrapper.js:131:4)
at IncomingMessage.emit (events.js:117:20)
at _stream_readable.js:929:16
at process._tickCallback (node.js:419:13)

Btw i already have jslint.js, don't know if it can help to understand.

This small change to formatOutput in wrapper.js changes output formatting so that in VisualStudio, you can double-click on an warning/error to jump to the line in the code.

//var line = [filepath, error.line, error.character, error.reason].join(":");
// format for VisualStudio
var line = [ filepath + "(" + error.line + ")", error.character, error.reason].join(":");

how do I run the script so that it accepts $ function?
currently

$(document).ready(function(data) {
});

throws this error:

inventory.js:92:1:'$' was used before it was defined.
inventory.js:92:3:'document' was used before it was defined.

I tried predefined=[$], predefined=['$'], predefined=$, predefined='$', predefined='[$]' and a few others (without really knowing what I'm doing).

Dear FND

I'm using the jslint-reporter with node.js on windows and ran into two problems.

1. You use util.print for printing the errors (line 82) but stderr.write for everthing else. When using the wrapper from the console, this works fine, but running it as a subprocess from a python script I cannot access the console output of util.print. Changing it to stderr.write works fine.

2. process.stdout.flush() (line 187) results in an error on widows. commenting it out works fine. maybe you could find a way to make that work on all systems

Thank you for this very useful script!

Cheers
Georg

jslint-reporter seems to truncate output to the last 10 errors, whatever the option is for jslint.

The problem seems to be with the util.print function in wrapper.js.

Replacing

util.print(errors.join("\n") + "\n");

with
for(i=0;i<errors.length;++i) {
util.print(errors[i]);
util.print("\n");
}

seems to solve the problem.