jshint reports failure on javascript files and should be fixed. Whenever this is done, one can make the jslint job to be voting by editing integration/zuul-config.git layout.yaml file and remove the line that make the jslint job to be non voting.
Version: master
Severity: normal
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Paladox | T62619 All repositories should pass jshint test | |||
| Resolved | Wikinaut | T63621 RSS should pass jshint |
(In reply to Antoine "hashar" Musso from comment #0)
jshint reports failure on javascript files and should be fixed. Whenever
this is done, one can make the jslint job to be voting by editing
integration/zuul-config.git layout.yaml file and remove the line that make
the jslint job to be non voting.
Can you give send me link, or screenshot, what exactly you mean ?
Change 114526 had a related patch set uploaded by Hashar:
mwext-RSS-jslint has no .js, make it voting
The RSS extensions has no javascript file, hence the Jenkins job http://integration.wikimedia.org/ci/job/mwext-RSS-jslint/? (which runs jshint) never got triggered.
I made it to vote, so whenever a Javascript file is added and does not pass jshint it will vote Verified -1.
Thank you and sorry for the trouble.