When there are errors in the file code , why am I still getting +1 as code review in gerrit

[karthikvee]

I am testing the following code in a file test.js with jshint like $jshint test.js:-
function myFunction(p1, p2) { ; ; return p1 == p2; // The function returns the product of p1 and p2 }
and Iam getting following error

test.js: line 2, col 1, Unnecessary semicolon.
test.js: line 3, col 2, Unnecessary semicolon.
2 errors

But while testing it using sputnik on gerrit, sputnik is giving +1 for it. When there are errors in the file , why am I still getting +1 as code review in gerrit

[pjagielski]

Hi, can you please attach Sputnik execution logs?

[karthikvee]

@pjagielski Here in this file I've attached Execution Logs:

output-sputnik-log.txt

[pjagielski]

So in this log Sputnik complains about no jshint.json configuration, but there nothing regarding JSLint. Do you have jslint.enabled=true in sputnik.properties

[karthikvee]

@pjagielski No, jslint.enabled=false and Do I have to manually create jshint.json configuration, sputnik is supposed to take care of the configuration of PMD, Jshint etc because they are inbuilt right? Or

Do we have to manually configure them [PMD, Jshint, Checkstyle, Codenarc etc]?

The contents of my sputnik directory are "bin, gerrit.properties, lib"

I'm assuming when you are saying "sputnik.properties" file as "gerrit.properties" file which I am using under sputnik directory here Iam attaching it.

Sample Config: gerrit_properties.txt

[SpOOnman]

@karthikvee did you manage to solve this configuration problem?