Merge pull request #434 from nschonni/jshint-cleanup

Jshint cleanup

Author: nschonni

.jshintignore

@@ -0,0 +1,4 @@
+node_modules/
+release/
+lib/
+build/

.jshintrc

@@ -0,0 +1,18 @@
+{
+    "curly": true,
+    "immed": true,
+    "latedef": true,
+    "newcap": true,
+    "noarg": true,
+    "sub": true,
+    "undef": true,
+    "boss": true,
+    "eqnull": true,
+    "forin": true,
+    "noempty": true,
+    "rhino": false,
+    "jquery": true,
+    "globals": {
+        "CSSLint": true
+    }
+}

Gruntfile.js

@@ -156,24 +156,7 @@ module.exports = function(grunt) {
         },
         jshint: {
             options: {
-                curly: true,
-                //eqeqeq: true, TODO: Leaving off due to errors
-                immed: true,
-                latedef: true,
-                newcap: true,
-                noarg: true,
-                sub: true,
-                undef: true,
-                //unused: true, TODO: Leaving off due to errors around unused instances of CSSLint
-                boss: true,
-                eqnull: true,
-                // Copied from build.xml
-                forin: true,
-                noempty: true,
-                rhino: false,
-                globals: {
-                    jQuery: true
-                }
+                jshintrc: ".jshintrc"
             },
             gruntfile: {
                 src: 'Gruntfile.js'

package.json

@@ -41,12 +41,12 @@
   },
   "devDependencies": {
     "grunt": "~0.4.1",
-    "grunt-contrib-jshint": "~0.5.4",
-    "grunt-contrib-concat": "~0.3.0",
-    "grunt-contrib-watch": "~0.4.4",
-    "yuitest": "~0.7.9",
     "grunt-contrib-clean": "~0.4.1",
+    "grunt-contrib-concat": "~0.3.0",
     "grunt-contrib-copy": "~0.4.1",
-    "grunt-include-replace": "~0.4.0"
+    "grunt-contrib-jshint": "~0.10.0",
+    "grunt-contrib-watch": "~0.4.4",
+    "grunt-include-replace": "~0.4.0",
+    "yuitest": "~0.7.9"
   }
 }

src/cli/common.js

@@ -2,7 +2,7 @@
  * Encapsulates all of the CLI functionality. The api argument simply
  * provides environment-specific functionality.
  */
-/*global CSSLint*/
+
 function cli(api){
 
     var globalOptions = {

src/core/Reporter.js

@@ -1,4 +1,3 @@
-/*global CSSLint*/
 /**
  * An instance of Report is used to report results of the
  * verification back to the main API.
@@ -158,4 +157,4 @@ Reporter.prototype = {
 };
 
 //expose for testing purposes
-CSSLint._Reporter = Reporter;
+CSSLint._Reporter = Reporter;

src/core/Util.js

@@ -1,6 +1,3 @@
-
-/*global CSSLint*/
-
 /*
  * Utility functions that make life easier.
  */
@@ -59,4 +56,4 @@ CSSLint.Util = {
             }
         }
     }
-};
+};

src/formatters/checkstyle-xml.js

@@ -1,4 +1,3 @@
-/*global CSSLint*/
 (function() {
 
     /**
@@ -106,4 +105,4 @@
         }
     });
 
-}());
+}());

src/formatters/compact.js

@@ -1,4 +1,3 @@
-/*global CSSLint*/
 CSSLint.addFormatter({
     //format information
     id: "compact",

src/formatters/csslint-xml.js

@@ -1,4 +1,3 @@
-/*global CSSLint*/
 CSSLint.addFormatter({
     //format information
     id: "csslint-xml",
@@ -65,4 +64,4 @@ CSSLint.addFormatter({
 
         return output.join("");
     }
-});
+});