MW has coding conventions for JavaScript (https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript)
During GSOC, I deliberately reviewed for some of these issues, but not all. Now, however, the code should be updated to fully comply.
In addition to the items listed there, it should pass jshint (a tool that checks code standards) and there should be a voting Jenkins job for this (so once it passes it stays that way).
Version: unspecified
Severity: normal
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| build: Enable jshint and jscs jobs in npm, and make pass | mediawiki/extensions/PronunciationRecording | master | +136 -92 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Paladox | T62619 All repositories should pass jshint test | |||
| Resolved | hashar | T56907 PronunciationRecording should pass jshint and comply with coding conventions |
Change 233095 had a related patch set uploaded (by Ricordisamoa):
Fix many jshint and jscs errors
Change 233095 merged by jenkins-bot:
build: Enable jshint and jscs jobs in npm, and make pass