Add clean task, add minified and non-minified CSS in production

It also may make sense to have a non-minified JS in production because those using build tools
will probably be minifying their production builds, but not minifying their dev builds, so it'd be easier
for them to have a non-minified version.
Remove version number from dev CSS file

Author: ray-print

Gruntfile.js

@@ -1,5 +1,5 @@
 //TODO: During next major version bump change to /dist. Leaving at ./form-validator for backwards
-//compabilitiy
+//compatibility
 const DIST_DIR = './form-validator';
 const MAIN_PLUGIN_FILE = 'form-validator/jquery.form-validator.min.js';
 const SRC_DIR = './src';
@@ -8,78 +8,23 @@ const MODULE_DIR = '/modules/';
 const LANG_DIR = '/lang/';
 const CSS_FILE = 'theme-default.css';
 
-
 var fs = require('fs'),
-  filesToBuild = {
-    uglify: {},
-    concat: {
-      main:{
-        src: [SRC_DIR + MAIN_DIR+'core-validators.js'],
-        dest: MAIN_PLUGIN_FILE
-      }
-    }
-  },
   readFile = function (file) {
     return fs.readFileSync(file, 'utf-8');
   },
   replaceInFile = function (path, from, to) {
     fs.writeFileSync(path, readFile(path).replace(from, to));
   };
 
-module.exports = function (grunt) {
-  // Gather up all module and language files
-  [MODULE_DIR, LANG_DIR].forEach(function (path) {
-    var srcPath = SRC_DIR + path;
-    var distPath = DIST_DIR + path;
-    if (path === MODULE_DIR) {
-      distPath = DIST_DIR + '/';
-    }
-
-    fs.readdirSync(srcPath).forEach(function (fileName) {
-        var fullPath = srcPath + fileName;
-        filesToBuild.uglify[distPath + fileName] = fullPath;
-        filesToBuild.concat[fullPath] = {
-          src: [fullPath],
-          dest: distPath + fileName
-        };
-    });
-  });
-
-  // Gather up all source files that will added to minified core library
-  fs.readdirSync(SRC_DIR + MAIN_DIR).forEach(function (fileName) {
-    var fullPath = SRC_DIR + MAIN_DIR + fileName;
-    if (filesToBuild.concat.main.src.indexOf(fullPath) === -1) {
-      filesToBuild.concat.main.src.unshift(fullPath);
-    }
-  });
-
-  filesToBuild.cssFiles = [];
-  filesToBuild.cssFiles.push({
-    dest: DIST_DIR,
-    src: CSS_FILE,
-    cwd: SRC_DIR,
-    expand: true
-  });
-
-  // Add options for concat and uglify
-  filesToBuild.concat.options = {
-    banner: "<%= meta.banner %>"
-  };
-  filesToBuild.uglify.options = {
-    banner: "<%= meta.banner %>"
-  };
-
-  // Add main script to uglify
-  filesToBuild.uglify[MAIN_PLUGIN_FILE] = MAIN_PLUGIN_FILE;
-
+function initializeGruntConfig(grunt, filesToBuild) {
   grunt.initConfig({
 
     // Import package manifest
     pkg: grunt.file.readJSON("package.json"),
 
     // Banner definitions
     meta: {
-      banner: "/**\n" +
+      banner: "/** File generated by Grunt -- do not modify\n" +
       " *  <%= (pkg.title || pkg.name).toUpperCase() %>\n" +
       " *\n" +
       " *  @version <%= pkg.version %>\n" +
@@ -93,9 +38,9 @@ module.exports = function (grunt) {
     concat: filesToBuild.concat,
 
     cssmin: {
-        target: {
-          files: filesToBuild.cssFiles
-        }
+      target: {
+        files: filesToBuild.cssFiles
+      }
     },
     // Lint definitions
     jshint: {
@@ -115,7 +60,7 @@ module.exports = function (grunt) {
     watch: {
       files: [SRC_DIR + '/**'],
       tasks: ['test'],
-      options : { nospawn : true }
+      options: { nospawn: true}
     },
 
     // Unit tests
@@ -132,10 +77,72 @@ module.exports = function (grunt) {
           keepalive: true
         }
       }
+    },
+
+    clean: [DIST_DIR + '/']
+  });
+}
+
+module.exports = function (grunt) {
+  var filesToBuild =  {
+      uglify: {},
+      concat: {
+        main:{
+          src: [SRC_DIR + MAIN_DIR+'core-validators.js'],
+          dest: MAIN_PLUGIN_FILE
+        }
+      }
+  };
+  // Gather up all module and language files
+  [MODULE_DIR, LANG_DIR].forEach(function (path) {
+    var srcPath = SRC_DIR + path;
+    var distPath = DIST_DIR + path;
+    if (path === MODULE_DIR) {
+      distPath = DIST_DIR + '/';
+    }
+
+    fs.readdirSync(srcPath).forEach(function (fileName) {
+        var fullPath = srcPath + fileName;
+        filesToBuild.uglify[distPath + fileName] = fullPath;
+        filesToBuild.concat[fullPath] = {
+          src: [fullPath],
+          dest: distPath + fileName
+        };
+    });
+  });
+  filesToBuild.concat[CSS_FILE] = {
+    src: [SRC_DIR + '/' + CSS_FILE],
+    dest: DIST_DIR + '/' + CSS_FILE
+  };
+  // Gather up all source files that will added to minified core library
+  fs.readdirSync(SRC_DIR + MAIN_DIR).forEach(function (fileName) {
+    var fullPath = SRC_DIR + MAIN_DIR + fileName;
+    if (filesToBuild.concat.main.src.indexOf(fullPath) === -1) {
+      filesToBuild.concat.main.src.unshift(fullPath);
     }
+  });
 
+  filesToBuild.cssFiles = [];
+  filesToBuild.cssFiles.push({
+    dest: DIST_DIR,
+    src: CSS_FILE,
+    cwd: SRC_DIR,
+    expand: true,
+    ext: '.min.css'
   });
 
+  // Add options for concat and uglify
+  filesToBuild.concat.options = {
+    banner: "<%= meta.banner %>"
+  };
+  filesToBuild.uglify.options = {
+    banner: "<%= meta.banner %>"
+  };
+
+  // Add main script to uglify
+  filesToBuild.uglify[MAIN_PLUGIN_FILE] = MAIN_PLUGIN_FILE;
+
+  initializeGruntConfig(grunt, filesToBuild);
   /*
    * Change to new version or the next version number. The project must be built again after this task
    * in order for the version change to take effect.
@@ -163,7 +170,7 @@ module.exports = function (grunt) {
     grunt.config.set('pkg', pkg);
   });
 
-
+  grunt.loadNpmTasks('grunt-contrib-clean');
   grunt.loadNpmTasks("grunt-contrib-concat");
   grunt.loadNpmTasks("grunt-contrib-jshint");
   grunt.loadNpmTasks("grunt-contrib-uglify");
@@ -174,6 +181,4 @@ module.exports = function (grunt) {
   grunt.registerTask("build-production", ["version", "cssmin", "test", "uglify"]);
   grunt.registerTask('test', ['concat', 'cssmin','jshint', 'qunit']);
   grunt.registerTask("default", ["test", "watch"]);
-  //TODO: add clean task, don't minify CSS in dev build, ?remove volo (its {version} is busted anyway)
-  //Add unminified CSS to prod build
 };

package.json

@@ -21,13 +21,11 @@
     "url": "http://victorjonsson.se"
   },
   "homepage": "http://formvalidator.net/",
-  "volo": {
-    "url": "https://raw.githubusercontent.com/lagden/jQuery-Form-Validator/{version}/form-validator/jquery.form-validator.js"
-  },
   "license": "MIT",
   "devDependencies": {
     "grunt": "~0.4.5",
     "grunt-cli": "~0.1.13",
+    "grunt-contrib-clean": "~1.0.0",
     "grunt-contrib-concat": "^0.5.1",
     "grunt-contrib-connect": "^0.11.2",
     "grunt-contrib-cssmin": "~0.14.0",