Skip to content

Commit

Permalink
Build: Don't run tests on grunt
Browse files Browse the repository at this point in the history
Running `grunt` will now just build Sizzle, do a size comparison & run ESLint,
similarly to what we do in jQuery. `grunt test` is used to run unit tests and
it requires running `grunt` or `grunt build` before to generate the built Sizzle
version.
  • Loading branch information
mgol committed Feb 28, 2020
1 parent 4dbaa59 commit b461b32
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
1 change: 0 additions & 1 deletion Gruntfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,6 @@ module.exports = function( grunt ) {
] );
grunt.registerTask( "default", [
"build",
"tests",
"compare_size",
"eslint:dist"
] );
Expand Down
2 changes: 1 addition & 1 deletion build/release.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ module.exports = function( Release ) {
issueTracker: "github",

generateArtifacts: function( done ) {
Release.exec( "grunt build", "Grunt command failed" );
Release.exec( "grunt", "Grunt command failed" );
done( files );
}
} );
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@
"scripts": {
"build": "npm install && grunt",
"start": "grunt start",
"test": "grunt test"
"test": "grunt && grunt test"
},
"commitplease": {
"components": [
Expand Down

0 comments on commit b461b32

Please sign in to comment.