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 | None | T63606 InputBox should pass jshint |
Change 120487 had a related patch set uploaded by 01tonythomas:
Adding a JS Class to inputBox to grey out Submit button.
(In reply to Antoine "hashar" Musso from comment #0)
What does JSHINT complain about and *where*? There is not a single JavaScript file in the repo currently [1]. Only JSON. Please be more specific.
[1] https://github.com/wikimedia/mediawiki-extensions-InputBox/tree/b85c6688
Change 125393 had a related patch set uploaded by Hashar:
Make mwext-InputBox-jslint voting
(In reply to Rainer Rillke @commons.wikimedia from comment #2)
What does JSHINT complain about and *where*? There is not a single
JavaScript file in the repo currently [1]. Only JSON. Please be more
specific.[1] https://github.com/wikimedia/mediawiki-extensions-InputBox/tree/b85c6688
I have mass filled bugs for any extension that had the jslint job not voting. I havent looked closely at each extension :-]
Since there is no javascript I have made the jslint job voting. The job also lint the i18n json files :-]
Thanks for the feedback!
Change 120487 merged by jenkins-bot:
Adding a JS Class to inputBox to grey out Submit button.