From: Michał Gołębiowski <m.goleb@gmail.com>
Date: Mon, 25 Jul 2016 23:22:53 +0000 (+0200)
Subject: Build: Don't lint every file in dist/ (#3245)
X-Git-Tag: 3.1.1~20
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=edf7a43681ca0ba517b155fd5d748921660dfcd2;p=jquery.git

Build: Don't lint every file in dist/ (#3245)

Currently the "all" target for the eslint task includes way more than
the "dev" & "dist" targets combined and those 2 tasks are the one run in
`npm test`.
---

diff --git a/Gruntfile.js b/Gruntfile.js
index 7af3ac98a..8cc4c91c0 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -113,7 +113,6 @@ module.exports = function( grunt ) {
 				// See https://github.com/sindresorhus/grunt-eslint/issues/119
 				quiet: true
 			},
-			all: ".",
 			dist: "dist/jquery.js",
 			dev: [ "src/**/*.js", "Gruntfile.js", "test/**/*.js", "build/**/*.js" ]
 		},
@@ -197,7 +196,8 @@ module.exports = function( grunt ) {
 
 	grunt.registerTask( "lint", [
 		"jsonlint",
-		runIfNewNode( "eslint:all" )
+		runIfNewNode( "eslint:dev" ),
+		runIfNewNode( "eslint:dist" )
 	] );
 
 	grunt.registerTask( "test_fast", [ runIfNewNode( "node_smoke_tests" ) ] );
@@ -225,6 +225,7 @@ module.exports = function( grunt ) {
 
 	grunt.registerTask( "precommit_lint", [
 		"newer:jsonlint",
-		runIfNewNode( "newer:eslint:all" )
+		runIfNewNode( "newer:eslint:dev" ),
+		runIfNewNode( "newer:eslint:dist" )
 	] );
 };