chore: migrate from grunt eslint to npm script (dequelabs#1247)

.eslintignore

@@ -1,4 +1,9 @@
+**/node_modules/**/*.js
+
 build/tasks/generate-imports.js
-!lib/core/imports/index.js
+build/tasks/aria-supported.js
+
+doc/api/*
+doc/examples/jest_react/*.js
+
 lib/core/imports/*.js
-doc/api/*

Gruntfile.js

@@ -13,12 +13,12 @@ module.exports = function(grunt) {
 	grunt.loadNpmTasks('grunt-contrib-concat');
 	grunt.loadNpmTasks('grunt-contrib-connect');
 	grunt.loadNpmTasks('grunt-contrib-copy');
-	grunt.loadNpmTasks('grunt-eslint');
 	grunt.loadNpmTasks('grunt-contrib-uglify');
 	grunt.loadNpmTasks('grunt-contrib-watch');
 	grunt.loadNpmTasks('grunt-mocha');
 	grunt.loadNpmTasks('grunt-parallel');
 	grunt.loadNpmTasks('grunt-markdownlint');
+	grunt.loadNpmTasks('grunt-run');
 	grunt.loadTasks('build/tasks');
 
 	var langs;
@@ -331,7 +331,6 @@ module.exports = function(grunt) {
 		mocha: testConfig(grunt, {
 			reporter: grunt.option('reporter') || 'Spec'
 		}),
-
 		connect: {
 			test: {
 				options: {
@@ -341,23 +340,10 @@ module.exports = function(grunt) {
 				}
 			}
 		},
-		eslint: {
-			axe: {
-				options: {
-					eslintrc: true,
-					reporter: grunt.option('report') ? 'checkstyle' : undefined,
-					reporterOutput: grunt.option('report') ? 'tmp/lint.xml' : undefined
-				},
-				src: [
-					'lib/**/*.js',
-					'test/**/*.js',
-					'build/**/*.js',
-					'doc/**/*.js',
-					'!doc/examples/jest_react/*.js',
-					'Gruntfile.js',
-					'!build/tasks/aria-supported.js',
-					'!**/node_modules/**/*.js'
-				]
+		run: {
+			npm_run_eslint: {
+				cmd: 'npm',
+				args: ['run', 'eslint']
 			}
 		},
 		markdownlint: {
@@ -372,10 +358,14 @@ module.exports = function(grunt) {
 
 	grunt.registerTask('default', ['build']);
 
-	grunt.registerTask('build', [
+	grunt.registerTask('pre-build', [
 		'clean',
 		'generate-imports',
-		'eslint',
+		'run:npm_run_eslint'
+	]);
+
+	grunt.registerTask('build', [
+		'pre-build',
 		'validate',
 		'concat:commons',
 		'configure',
@@ -393,7 +383,6 @@ module.exports = function(grunt) {
 		'connect',
 		'mocha',
 		'parallel',
-		'eslint',
 		'markdownlint'
 	]);
 
@@ -402,22 +391,19 @@ module.exports = function(grunt) {
 		'testconfig',
 		'fixture',
 		'connect',
-		'parallel',
-		'eslint'
+		'parallel'
 	]);
 
 	grunt.registerTask('test-fast', [
 		'build',
 		'testconfig',
 		'fixture',
 		'connect',
-		'mocha',
-		'eslint'
+		'mocha'
 	]);
 
 	grunt.registerTask('translate', [
-		'clean',
-		'eslint',
+		'pre-build',
 		'validate',
 		'concat:commons',
 		'add-locale'

package.json

@@ -57,6 +57,7 @@
   "scripts": {
     "api-docs": "jsdoc --configure .jsdoc.json",
     "build": "grunt",
+    "eslint": "eslint --format stylish '{lib,test,build,doc}/**/*.js' 'Gruntfile.js'",
     "test": "npm run retire && tsc && grunt test",
     "version": "echo \"use 'npm run release' to bump axe-core version\" && exit 1",
     "prepublishOnly": "grunt build && grunt file-exists",
@@ -89,10 +90,10 @@
     "grunt-contrib-copy": "^1.0.0",
     "grunt-contrib-uglify": "^4.0.0",
     "grunt-contrib-watch": "^1.1.0",
-    "grunt-eslint": "^21.0.0",
     "grunt-markdownlint": "^2.0.0",
     "grunt-mocha": "1.1.0",
     "grunt-parallel": "^0.5.1",
+    "grunt-run": "^0.8.1",
     "html-entities": "^1.2.0",
     "husky": "^1.1.1",
     "jquery": "^3.0.0",