Build: Append .eslintignore paths to grunt eslint paths

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.

Closes gh-4689
This commit is contained in:
Ed Sanders 2020-04-21 12:43:52 +01:00 committed by Michał Gołębiowski-Owczarek
parent e7b3bc488d
commit a22b43bad4

View File

@ -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"
@ -88,7 +87,18 @@ module.exports = function( grunt ) {
src: [ "dist/jquery.js", "dist/jquery.min.js" ]
},
dev: {
src: [ "src/**/*.js", "Gruntfile.js", "test/**/*.js", "build/**/*.js" ]
src: [
"src/**/*.js",
"Gruntfile.js",
"test/**/*.js",
"build/**/*.js",
// Ignore files from .eslintignore
// See https://github.com/sindresorhus/grunt-eslint/issues/119
...new CLIEngine()
.getConfigForFile( "Gruntfile.js" )
.ignorePatterns.map( ( p ) => `!${ p }` )
]
}
},
testswarm: {