Commita22b43b

authored and

committed

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.Closesgh-4689

1 parente7b3bc4 commita22b43bCopy full SHA for a22b43b

File tree

+15

-5

lines changed

+15

-5

lines changed

Lines changed: 15 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,8 @@ module.exports = function( grunt ) {`
`15`	`15`	`varfs=require("fs"),`
`16`	`16`	`gzip=require("gzip-js"),`
`17`	`17`	`isTravis=process.env.TRAVIS,`
`18`		`-travisBrowsers=process.env.BROWSERS&&process.env.BROWSERS.split(",");`
	`18`	`+travisBrowsers=process.env.BROWSERS&&process.env.BROWSERS.split(","),`
	`19`	`+CLIEngine=require("eslint").CLIEngine;`
`19`	`20`
`20`	`21`	`if(!grunt.option("filename")){`
`21`	`22`	`grunt.option("filename","jquery.js");`
`@@ -77,9 +78,7 @@ module.exports = function( grunt ) {`
`77`	`78`	`},`
`78`	`79`	`eslint:{`
`79`	`80`	`options:{`
`80`		`-`
`81`		`-// See https://github.com/sindresorhus/grunt-eslint/issues/119`
`82`		`-quiet:true`
	`81`	`+maxWarnings:0`
`83`	`82`	`},`
`84`	`83`
`85`	`84`	`// We have to explicitly declare "src" property otherwise "newer"`
`@@ -88,7 +87,18 @@ module.exports = function( grunt ) {`
`88`	`87`	`src:["dist/jquery.js","dist/jquery.min.js"]`
`89`	`88`	`},`
`90`	`89`	`dev:{`
`91`		`-src:["src/*/.js","Gruntfile.js","test/*/.js","build/*/.js"]`
	`90`	`+src:[`
	`91`	`+"src/*/.js",`
	`92`	`+"Gruntfile.js",`
	`93`	`+"test/*/.js",`
	`94`	`+"build/*/.js",`
	`95`	`+`
	`96`	`+// Ignore files from .eslintignore`
	`97`	`+// See https://github.com/sindresorhus/grunt-eslint/issues/119`
	`98`	`+...newCLIEngine()`
	`99`	`+.getConfigForFile("Gruntfile.js")`
	`100`	+.ignorePatterns.map((p)=>`!${p}`)
	`101`	`+]`
`92`	`102`	`}`
`93`	`103`	`},`
`94`	`104`	`testswarm:{`

Comments

(0)