Jshint cleanup

.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("");
     }
-});
+});

src/formatters/junit-xml.js

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

src/formatters/lint-xml.js

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

src/formatters/text.js

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

src/rules/adjoining-classes.js

@@ -1,7 +1,7 @@
-/*global CSSLint*/
 /*
  * Rule: Don't use adjoining classes (.foo.bar).
  */
+
 CSSLint.addRule({
 
     //rule information
@@ -42,4 +42,4 @@ CSSLint.addRule({
         });
     }
 
-});
+});

src/rules/box-model.js

@@ -1,5 +1,3 @@
-/*global CSSLint*/
-
 /*
  * Rule: Don't use width or height when using padding or border.
  */
@@ -98,4 +96,4 @@ CSSLint.addRule({
         parser.addListener("endkeyframerule", endRule);
     }
 
-});
+});

src/rules/box-sizing.js

@@ -1,8 +1,7 @@
-/*global CSSLint*/
-
 /*
  * Rule: box-sizing doesn't work in IE6 and IE7.
  */
+
 CSSLint.addRule({
 
     //rule information
@@ -25,4 +24,4 @@ CSSLint.addRule({
         });
     }
 
-});
+});

src/rules/bulletproof-font-face.js

@@ -2,7 +2,7 @@
  * Rule: Use the bulletproof @font-face syntax to avoid 404's in old IE
  * (http://www.fontspring.com/blog/the-new-bulletproof-font-face-syntax)
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information
@@ -63,4 +63,4 @@ CSSLint.addRule({
             }
         });
     }
-});
+});

src/rules/compatible-vendor-prefixes.js

@@ -2,7 +2,7 @@
  * Rule: Include all compatible vendor prefixes to reach a wider
  * range of users.
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information
@@ -183,4 +183,4 @@ CSSLint.addRule({
             }
         });
     }
-});
+});

src/rules/display-property-grouping.js

@@ -5,7 +5,7 @@
  * - vertical-align should not be used with block
  * - margin, float should not be used with table-*
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information
@@ -114,4 +114,4 @@ CSSLint.addRule({
 
     }
 
-});
+});

src/rules/duplicate-background-images.js

@@ -1,7 +1,7 @@
 /*
  * Rule: Disallow duplicate background-images (using url).
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information
@@ -34,4 +34,4 @@ CSSLint.addRule({
             }
         });
     }
-});
+});

src/rules/duplicate-properties.js

@@ -2,7 +2,7 @@
  * Rule: Duplicate properties must appear one after the other. If an already-defined
  * property appears somewhere else in the rule, then it's likely an error.
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information
@@ -43,4 +43,4 @@ CSSLint.addRule({
 
     }
 
-});
+});

src/rules/empty-rules.js

@@ -1,7 +1,7 @@
 /*
  * Rule: Style rules without any properties defined should be removed.
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information
@@ -31,4 +31,4 @@ CSSLint.addRule({
         });
     }
 
-});
+});

src/rules/errors.js

@@ -1,7 +1,7 @@
 /*
  * Rule: There should be no syntax errors. (Duh.)
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information
@@ -20,4 +20,4 @@ CSSLint.addRule({
 
     }
 
-});
+});

src/rules/fallback-colors.js

@@ -1,5 +1,3 @@
-
-/*global CSSLint*/
 CSSLint.addRule({
 
     //rule information
@@ -74,4 +72,4 @@ CSSLint.addRule({
 
     }
 
-});
+});

src/rules/floats.js

@@ -2,7 +2,7 @@
  * Rule: You shouldn't use more than 10 floats. If you do, there's probably
  * room for some abstraction.
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information
@@ -33,4 +33,4 @@ CSSLint.addRule({
         });
     }
 
-});
+});

src/rules/font-faces.js

@@ -1,7 +1,7 @@
 /*
  * Rule: Avoid too many @font-face declarations in the same stylesheet.
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information
@@ -27,4 +27,4 @@ CSSLint.addRule({
         });
     }
 
-});
+});

src/rules/font-sizes.js

@@ -2,7 +2,6 @@
  * Rule: You shouldn't need more than 9 font-size declarations.
  */
 
-/*global CSSLint*/
 CSSLint.addRule({
 
     //rule information
@@ -32,4 +31,4 @@ CSSLint.addRule({
         });
     }
 
-});
+});

src/rules/gradients.js

@@ -1,7 +1,7 @@
 /*
  * Rule: When using a vendor-prefixed gradient, make sure to use them all.
  */
-/*global CSSLint*/
+
 CSSLint.addRule({
 
     //rule information