Page MenuHomePhabricator

SemanticForms should pass jshint
Closed, ResolvedPublic

Description

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

Details

Reference
bz61630

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:53 AM
bzimport set Reference to bz61630.

Change 151970 had a related patch set uploaded by Jatin:
Making SF pass JSHint test

https://gerrit.wikimedia.org/r/151970

Change 151970 merged by Yaron Koren:
Making SF pass JSHint test

https://gerrit.wikimedia.org/r/151970

hashar - can you confirm if this is now fixed?

Change 160623 had a related patch set uploaded by Hashar:
SemanticForms now pass jshint make it voting

https://gerrit.wikimedia.org/r/160623

Change 160623 merged by jenkins-bot:
SemanticForms now pass jshint make it voting

https://gerrit.wikimedia.org/r/160623

Thanks Yaron! It even has a .jshintrc file.

I have made the Jenkins job voting :-)