From 2330848ddad077bebb5cd83c83b229cec2970e15 Mon Sep 17 00:00:00 2001 From: Richard Herrera Date: Wed, 11 Dec 2013 08:36:16 -0800 Subject: [PATCH] A clean up of some recent activity regarding dependencies. See https://github.com/Modernizr/Modernizr/pull/1159 These dependencies are only used for testing. They should be moved to devDependencies. The benefits for this include a faster install time, and suppressing the `>> Local Npm module "grunt-contrib-qunit" not found` errors references in Issue #1154 Removing qunit from the default task, since it is now a conditional dev dependency. --- Gruntfile.js | 4 ++-- package.json | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Gruntfile.js b/Gruntfile.js index 79fa0ee1a0..a9aad19392 100644 --- a/Gruntfile.js +++ b/Gruntfile.js @@ -213,7 +213,7 @@ module.exports = function( grunt ) { }); // Load required contrib packages - require('matchdep').filterAll(['grunt-*', '!grunt-cli']).forEach(grunt.loadNpmTasks); + require('matchdep').filter(['grunt-*', '!grunt-cli']).forEach(grunt.loadNpmTasks); // devDependencies may or may not be installed require('matchdep').filterDev('grunt-*').forEach(function (contrib) { @@ -258,5 +258,5 @@ module.exports = function( grunt ) { // Build grunt.registerTask('build', ['clean', 'generateinit', 'requirejs', 'copy', 'clean:postbuild', 'stripdefine', 'uglify']); - grunt.registerTask('default', ['build', 'jshint', 'qunit']); + grunt.registerTask('default', ['build', 'jshint']); }; diff --git a/package.json b/package.json index 2a7d739edf..96239c2d7e 100644 --- a/package.json +++ b/package.json @@ -20,11 +20,11 @@ "grunt-contrib-clean": "~0.4.0", "grunt-contrib-copy": "~0.4.0", "grunt-contrib-requirejs": "~0.4.0", - "grunt-contrib-connect": "~0.5.0", - "grunt-saucelabs": "~4.0.4", "grunt-cli": "~0.1.11" }, "devDependencies": { + "grunt-contrib-connect": "~0.5.0", + "grunt-saucelabs": "~4.0.4", "grunt-contrib-qunit": "~0.2.0", "grunt-contrib-nodeunit": "~0.2.0" },