[Bug 58279] Add a JSON linter to CI

2014-01-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 Gerrit Notification Bot gerritad...@wikimedia.org changed: What|Removed |Added Status|NEW

[Bug 58279] Add a JSON linter to CI

2014-01-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #9 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 107163 had a related patch set uploaded by Hashar: json-lint.php: recursively lint json files with PHP https://gerrit.wikimedia.org/r/107163 -- You are

[Bug 58279] Add a JSON linter to CI

2014-01-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 Antoine hashar Musso has...@free.fr changed: What|Removed |Added Status|PATCH_TO_REVIEW |ASSIGNED

[Bug 58279] Add a JSON linter to CI

2014-01-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #10 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 107163 had a related patch set uploaded by Krinkle: json-lint.php: recursively lint json files with PHP https://gerrit.wikimedia.org/r/107163 -- You are

[Bug 58279] Add a JSON linter to CI

2014-01-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 Gerrit Notification Bot gerritad...@wikimedia.org changed: What|Removed |Added Status|ASSIGNED

[Bug 58279] Add a JSON linter to CI

2014-01-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #11 from Krinkle krinklem...@gmail.com --- (In reply to comment #4) json files requires double quotes which jshint complains about. So I guess we need a different job that would use jshint with a specific configuration. Have you

[Bug 58279] Add a JSON linter to CI

2014-01-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #7 from Siebrand Mazeland siebr...@wikimedia.org --- I have an example of invalid JSON that got merged, and caused issues in the L10n process: Line 215 of https://gerrit.wikimedia.org/r/#/c/106460/1/data/i18n/qqq.json has a

[Bug 58279] Add a JSON linter to CI

2014-01-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #8 from Antoine hashar Musso has...@free.fr --- I guess one can write a basic lint script that would wrap around PHP json_decode() ? If we can decode it, I guess we are safe :] -- You are receiving this mail because: You are the

[Bug 58279] Add a JSON linter to CI

2014-01-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #2 from Siebrand Mazeland s.mazel...@xs4all.nl --- Any updates on this yet? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.

[Bug 58279] Add a JSON linter to CI

2014-01-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #3 from Antoine hashar Musso has...@free.fr --- The jslint job is already present for extension and is triggered by Zuul whenever a .jshint*, *.js or *.json file is modified. A culprit is that by default jshint does not parse .json

[Bug 58279] Add a JSON linter to CI

2014-01-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #4 from Antoine hashar Musso has...@free.fr --- json files requires double quotes which jshint complains about. So I guess we need a different job that would use jshint with a specific configuration. -- You are receiving this mail

[Bug 58279] Add a JSON linter to CI

2014-01-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #5 from Siebrand Mazeland s.mazel...@xs4all.nl --- (In reply to comment #4) json files requires double quotes which jshint complains about. So I guess we need a different job that would use jshint with a specific configuration.

[Bug 58279] Add a JSON linter to CI

2014-01-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #6 from Siebrand Mazeland s.mazel...@xs4all.nl --- (In reply to comment #5) (In reply to comment #4) json files requires double quotes which jshint complains about. So I guess we need a different job that would use jshint with

[Bug 58279] Add a JSON linter to CI

2013-12-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=58279 --- Comment #1 from Antoine hashar Musso has...@free.fr --- Timo thoughts? Should we incorporate that in the jslint job that runs jshint ? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for