Build: Append .eslintignore paths to grunt eslint paths

edg2s · edg2s · commit 2882f19f04e4 · 2020-04-21T12:45:09.000+01:00
This allows us to turn off the `quiet` option which was suppressing warnings.

We can also set `maxWarnings` to 0 now that aren't any.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -15,7 +15,8 @@ module.exports = function( grunt ) {
 	var fs = require( "fs" ),
 		gzip = require( "gzip-js" ),
 		isTravis = process.env.TRAVIS,
-		travisBrowsers = process.env.BROWSERS && process.env.BROWSERS.split( "," );
+		travisBrowsers = process.env.BROWSERS && process.env.BROWSERS.split( "," ),
+		CLIEngine = require( "eslint" ).CLIEngine;
 
 	if ( !grunt.option( "filename" ) ) {
 		grunt.option( "filename", "jquery.js" );
@@ -77,9 +78,7 @@ module.exports = function( grunt ) {
 		},
 		eslint: {
 			options: {
-
-				// See https://github.com/sindresorhus/grunt-eslint/issues/119
-				quiet: true
+				maxWarnings: 0
 			},
 
 			// We have to explicitly declare "src" property otherwise "newer"
@@ -89,6 +88,14 @@ module.exports = function( grunt ) {
 			},
 			dev: {
 				src: [ "src/**/*.js", "Gruntfile.js", "test/**/*.js", "build/**/*.js" ]
+
+					// Ignore files from .eslintignore
+					// See https://github.com/sindresorhus/grunt-eslint/issues/119
+					.concat(
+						new CLIEngine()
+							.getConfigForFile( "Gruntfile.js" )
+							.ignorePatterns.map( p => "!" + p )
+					)
 			}
 		},
 		testswarm: {
