Page MenuHomePhabricator

Spreadsheet 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: unspecified
Severity: normal

Related Objects

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 22 2014, 3:23 AM
bzimport set Reference to bz63668.
bzimport added a subscriber: Unknown Object (MLST).

Where is the bugtracker for [[mw:Extension:Spreadsheet]]? Here in Bugzilla a maintainer will never find out about this bug report if no maintainer was CC'ed by the reporter.

Addshore: Could you answer comment 1 please?

The repo is hosted on gerrit.
There is no individual component for the repo on bugzilla.
There is no other bugtracker listed.
Thus it is under "Mediawiki Extension" > [other]

Change 263069 had a related patch set uploaded (by Paladox):
Add npm and composer-test tests

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

hashar claimed this task.
hashar reassigned this task from hashar to Paladox.
hashar set Security to None.

Change 263069 merged by jenkins-bot:
Add npm and composer-test tests

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