Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
3.5.7, 3.6.5, 3.7.1, 3.8
-
MOODLE_35_STABLE, MOODLE_36_STABLE, MOODLE_37_STABLE, MOODLE_38_STABLE
-
MOODLE_35_STABLE, MOODLE_36_STABLE, MOODLE_37_STABLE
-
MDL-66265-master -
Description
I discovered in MDL-66192 that jshint is applying to all JS by default, even JS which we expect to be checked by eslint.
We only use jshint for YUI, and it is used intrinsically by Shifter. We can't just replace it with eslint sadly.
We are currently on version 0.9.1 of jshint, which is ancient (7+ years). That version does not support any way to either specify or ignore files.
I propose we upgrade to the latest version of jshint, which does support a .jshintignore and then ignore '*/amd/*'.
Attachments
Issue Links
- is blocked by
-
MDL-66532 Add .nvmrc to ensure it is consistent
-
- Closed
-