Page MenuHomePhabricator

JsonData should pass jshint
Closed, DeclinedPublic

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

Related Objects

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:51 AM
bzimport set Reference to bz61607.
bzimport added a subscriber: Unknown Object (MLST).

What is JsonData? Where is it? Who maintains it? Nobody on CC here, and
https://www.mediawiki.org/wiki/Extension:JsonData is empty.

Change 271512 had a related patch set uploaded (by Paladox):
Fix jslint errors

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

Change 271525 had a related patch set uploaded (by Paladox):
[JsonData] Replace jslint with jsonlint and jshint

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

Change 271525 abandoned by Hashar:
[JsonData] Replace jslint with jsonlint and jshint

Reason:
no more needed

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

The repository is no more actively developed, there is little point in spending time enforcing jshint there. At least the repository has been setup with the npm entry point.

Change 271512 abandoned by Paladox:
Add npm entry point and fix jshint errors

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