From edf7a43681ca0ba517b155fd5d748921660dfcd2 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Micha=C5=82=20Go=C5=82=C4=99biowski?= Date: Tue, 26 Jul 2016 01:22:53 +0200 Subject: [PATCH] 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`. --- Gruntfile.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) 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" ) ] ); }; -- 2.39.5