Add opinionated jshintrc - and point Grunt at it.

Author: dceejay

.gitignore

@@ -1,6 +1,5 @@
 .npm
 .jshintignore
-.jshintrc
 .project
 .settings
 .tern-project

.jshintrc

@@ -0,0 +1,13 @@
+{
+    "asi": true,        // allow missing semicolons
+    "curly": true,      // require braces
+    "eqnull": true,     // ignore ==null
+    "forin": true,      // require property filtering in "for in" loops
+    "immed": true,      // require immediate functions to be wrapped in ( )
+    "nonbsp": true,     // warn on unexpected whitespace breaking chars
+    //"strict": true,   // commented out for now as it causes 100s of warnings, but want to get there eventually
+    //"unused": true,   // Check for unused functions and variables
+    "loopfunc": true,   // allow functions to be defined in loops
+    //"expr": true,       // allow ternery operator syntax...
+    "sub": true         // don't warn that foo['bar'] should be written as foo.bar
+}

Gruntfile.js

@@ -15,7 +15,7 @@
  **/
 
 module.exports = function(grunt) {
-    
+
     // Project configuration.
     grunt.initConfig({
         pkg: grunt.file.readJSON('package.json'),
@@ -31,22 +31,23 @@ module.exports = function(grunt) {
         },
         jshint: {
             options: {
+                jshintrc:true     // Use external jshinrc file configured as below
                 // http://www.jshint.com/docs/options/
-                "asi": true,      // allow missing semicolons
-                "curly": true,    // require braces
-                "eqnull": true,   // ignore ==null
-                "forin": true,    // require property filtering in "for in" loops
-                "immed": true,    // require immediate functions to be wrapped in ( )
-                "nonbsp": true,   // warn on unexpected whitespace breaking chars
-                //"strict": true, // commented out for now as it causes 100s of warnings, but want to get there eventually
-                "loopfunc": true, // allow functions to be defined in loops
-                "sub": true       // don't warn that foo['bar'] should be written as foo.bar
+                //"asi": true,      // allow missing semicolons
+                //"curly": true,    // require braces
+                //"eqnull": true,   // ignore ==null
+                //"forin": true,    // require property filtering in "for in" loops
+                //"immed": true,    // require immediate functions to be wrapped in ( )
+                //"nonbsp": true,   // warn on unexpected whitespace breaking chars
+                ////"strict": true, // commented out for now as it causes 100s of warnings, but want to get there eventually
+                //"loopfunc": true, // allow functions to be defined in loops
+                //"sub": true       // don't warn that foo['bar'] should be written as foo.bar
             },
             all: {
                 src: ['*/*.js'],
                 filter: function(filepath) { // on some developer machines the test coverage HTML report utilities cause further failures
                     if(filepath.indexOf("coverage/prettify.js") === -1) {
-                        return true;   
+                        return true;
                     } else {
                         console.log("Filtered out " + filepath + " from the jshint checks");
                         return false;