I worked this out myself: I created a .jshintignore file \o/
https://gerrit.wikimedia.org/r/#/c/36919/2/.jshintignore
On Tue, Dec 4, 2012 at 1:22 PM, Jon Robson jdlrobson@gmail.com wrote:
This is now running on MobileFrontend [1] but needs some tweaking! It's awesome! Kudos to whoever enabled that.
Is it possible to blacklist certain files? For instance we have an external javascript file we pull in from another project (javascripts/externals/eventlog.js ) that I'd rather not run through jshint
What's the best way to do this?
[1] https://integration.mediawiki.org/ci/job/mwext-MobileFrontend-jslint/6/conso...
On Tue, Nov 20, 2012 at 2:27 PM, Krinkle krinklemail@gmail.com wrote:
Hey all,
For a while now we have .jshintrc rules in the repository and are able to run node-jshint locally.
TL;DR: jshint is now running from Jenkins on mediawiki/core (joining the linting sequence for php and puppet files).
I cleaned up the last old lint failures in the repo yesterday in preparation to enable it from Jenkins (like we already do for PHP and Puppet files). After some quick testing in a sandbox job on Jenkins to confirm it passes/fails accordingly, this has now been enabled in the main Jenkins job for mediawiki/core.
Right now only master and REL1_20 pass (REL1_19 and wmf branches do not, the next wmf branch will however pass).
Therefore is has only been enabled on the master branch for now.
Example success:
- https://gerrit.wikimedia.org/r/#/c/24249/
- https://integration.mediawiki.org/ci/job/MediaWiki-GIT-Fetching/7730/console
22:16:41 Running "jshint" task 22:16:48 OK 22:16:48 22:16:48 Done, without errors.
Example failure:
- https://gerrit.wikimedia.org/r/#/c/34433/
- https://integration.mediawiki.org/ci/job/MediaWiki-GIT-Fetching/7732/console
22:24:01 Running "jshint" task 22:24:08 >> resources/mediawiki/mediawiki.js: line 5, col 5, Identifier 'bla_bla' is not in camel case. 22:24:08 >> resources/mediawiki/mediawiki.js: line 5, col 12, 'bla_bla' is defined but never used. 22:24:08 >> 22:24:08 >> 2 errors 22:24:08 Warning: Task "jshint" failed.
So if your commit is marked as failure, just like with failures from phplint, puppetlint or phpunit: Click the link from jenkins-bot and follow the trail.
-- Timo Tijhof
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l
-- Jon Robson http://jonrobson.me.uk @rakugojon