MDL-52127 js: check amd files with eslint grunt task
authorDan Poltawski <dan@moodle.com>
Wed, 11 May 2016 09:19:30 +0000 (10:19 +0100)
committerDan Poltawski <dan@moodle.com>
Sat, 11 Jun 2016 09:14:35 +0000 (10:14 +0100)
commit3adb62b72785c6aa65106f6a93c2e32de55e3450
tree070850ea3b1d1bbfc3d6de78b4e79550f2de49d8
parent834f1b66d46df71ea418adfc5b0ba4b932239f65
MDL-52127 js: check amd files with eslint grunt task

I have spent quite a lot of time working through the current list of
eslint options and configuring them for Moodle style and I think this is
a very good basis to start us at (as well as taking some of out jshint
options out with https://www.npmjs.com/package/polyjuice ). Thanks to
Andrew Nicols, Mark Johnson and Frédéric Massart for some refinements.

With this configuration the grunt build will fail if errors are present
in the js (though you can of course tell jshint to ignore some errors,
as I have done in admin/tool/lp/amd/src/competency_rule_points.js and
defining the Y global in lib/amd/src/yui.js ).

The grunt task will not report warnings by default, but a new
--show-lint-warnings flag will help achieve that. Editor
integrations/stanadalone eslint tool will surely be a better way of
getting eslint errors rather than using the grunt task.
.eslintignore [new file with mode: 0644]
.eslintrc [new file with mode: 0644]
Gruntfile.js
admin/tool/lp/amd/src/competency_rule_points.js
lib/amd/src/localstorage.js
lib/amd/src/yui.js
npm-shrinkwrap.json
package.json