mirror of
https://github.com/jquery/jquery.git
synced 2024-11-23 02:54:22 +00:00
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`.
This commit is contained in:
parent
04a2e036d0
commit
edf7a43681
@ -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" )
|
||||
] );
|
||||
};
|
||||
|
Loading…
Reference in New Issue
Block a user