Last modified: 2014-09-22 03:22:01 UTC
When we started with jshint, we made the jslint jobs non voting by default. The reason was to enable reporting while developers are busy fixing their jshint errors. We eventually came out with a lot of exceptions to whitelist passing repositories. I believe it is time to reverse the logic and enforce jshint by default on all repositories.
Change 110385 had a related patch set uploaded by Hashar: make jslint voting by default https://gerrit.wikimedia.org/r/110385
Created attachment 14415 [details] list of jslint jobs non voting in Zuul Attached is a list of jslint jobs which are non voting as per https://gerrit.wikimedia.org/r/#/c/110385 We should verify whether it is currently passing, and if so make it voting, else have a bug filled so the project owners start fixing the jshint issues.
Change 110535 had a related patch set uploaded by Hashar: make some jslint voting https://gerrit.wikimedia.org/r/110535
Created attachment 14426 [details] list of jslint jobs non voting in Zuul (after change 110535)
Change 110385 merged by jenkins-bot: make jslint voting by default https://gerrit.wikimedia.org/r/110385
Change 110535 merged by jenkins-bot: make some jslint voting https://gerrit.wikimedia.org/r/110535
Change 114460 had a related patch set uploaded by Hashar: Non voting jshint jobs now have a bug report https://gerrit.wikimedia.org/r/114460
Change 114460 merged by jenkins-bot: Non voting jshint jobs now have a bug report https://gerrit.wikimedia.org/r/114460
Fix grammer in the title (either "Have all repositories pass" or "All repositories have to pass"; "Have all repositories to pass jshint test" is invalid)