Merge pull request #33 from twbs/lint

Remove JSHint and JSCS.

Author: cvrebert

.eslintrc

@@ -7,55 +7,103 @@
         "block-scoped-var": 2,
         "brace-style": [2, "stroustrup"],
         "camelcase": 2,
-        "comma-spacing": [2, {"before": false, "after": true}],
+        "comma-dangle": 2,
+        "comma-spacing": [2, { "before": false, "after": true }],
         "comma-style": [2, "last"],
+        "consistent-return": 2,
         "consistent-this": [2, "self"],
         "curly": 2,
         "eol-last": 2,
         "eqeqeq": 2,
+        "func-names": 0,
+        "func-style": [0, "expression"],
+        "guard-for-in": 2,
+        "handle-callback-err": 2,
         "indent": [2, 2],
-        "key-spacing": [2, {"beforeColon": false, "afterColon": true}],
+        "key-spacing": [2, { "beforeColon": false, "afterColon": true }],
         "keyword-spacing": 2,
         "new-cap": 2,
+        "newline-after-var": [0, "always"],
         "new-parens": 2,
         "no-array-constructor": 2,
         "no-bitwise": 2,
+        "no-cond-assign": 2,
         "no-constant-condition": 0,
+        "no-control-regex": 2,
+        "no-debugger": 2,
+        "no-delete-var": 2,
         "no-dupe-args": 2,
+        "no-dupe-keys": 2,
         "no-duplicate-case": 2,
+        "no-else-return": 2,
+        "no-empty": 2,
+        "no-empty-character-class": 2,
+        "no-eq-null": 2,
         "no-eval": 2,
-        "no-extra-strict": 0,
+        "no-ex-assign": 2,
+        "no-extra-boolean-cast": 2,
+        "no-extra-parens": 2,
+        "no-extra-semi": 2,
+        "no-fallthrough": 2,
         "no-floating-decimal": 2,
+        "no-func-assign": 2,
+        "no-implicit-coercion": 2,
         "no-implied-eval": 2,
         "no-inline-comments": 0,
+        "no-inner-declarations": 2,
+        "no-invalid-regexp": 2,
         "no-irregular-whitespace": 2,
+        "no-lone-blocks": 2,
+        "no-lonely-if": 2,
+        "no-loop-func": 2,
+        "no-mixed-requires": 2,
         "no-mixed-spaces-and-tabs": 2,
-        "no-multiple-empty-lines": 0,
+        "no-multiple-empty-lines": 2,
         "no-multi-spaces": 2,
+        "no-multi-str": 2,
+        "no-negated-in-lhs": 2,
         "no-new-object": 2,
+        "no-obj-calls": 2,
+        "no-octal": 2,
         "no-param-reassign": 0,
+        "no-path-concat": 2,
         "no-process-env": 2,
+        "no-redeclare": 2,
+        "no-regex-spaces": 2,
         "no-reserved-keys": 0,
+        "no-return-assign": 2,
         "no-self-compare": 2,
+        "no-sequences": 2,
         "no-spaced-func": 2,
+        "no-sparse-arrays": 2,
         "no-throw-literal": 2,
         "no-trailing-spaces": 2,
+        "no-undef": 2,
         "no-underscore-dangle": 0,
+        "no-unexpected-multiline": 2,
+        "no-unreachable": 2,
+        "no-unused-expressions": 2,
+        "no-unused-vars": 2,
+        "no-useless-call": 2,
+        "no-useless-concat": 2,
         "no-void": 2,
         "object-curly-spacing": [2, "never"],
         "one-var": [2, "never"],
         "operator-assignment": [2, "always"],
         "padded-blocks": 0,
-        "quotes": 0,
+        "quote-props": [2, "as-needed"],
+        "quotes": [2, "single"],
         "radix": 2,
         "semi": [2, "always"],
-        "semi-spacing": [2, {"before": false, "after": true}],
+        "semi-spacing": [2, { "before": false, "after": true }],
         "space-before-blocks": [2, "always"],
         "spaced-comment": [2, "always"],
         "space-in-parens": [2, "never"],
-        "space-unary-ops": [2, {"words": true, "nonwords": false}],
+        "space-unary-ops": [2, { "words": true, "nonwords": false }],
         "strict": [2, "global"],
+        "use-isnan": 2,
         "vars-on-top": 0,
+        "valid-typeof": 2,
         "wrap-iife": [2, "inside"],
         "yoda": [2, "never"]
     }

.jscsrc

@@ -12,17 +12,6 @@ module.exports = function (grunt) {
 
   // Project configuration.
   grunt.initConfig({
-    jshint: {
-      options: {
-        jshintrc: '.jshintrc'
-      },
-      all: [
-        'Gruntfile.js',
-        'tasks/*.js',
-        '<%= nodeunit.tests %>'
-      ]
-    },
-
     eslint: {
       options: {
         config: '.eslintrc'
@@ -34,17 +23,6 @@ module.exports = function (grunt) {
       ]
     },
 
-    jscs: {
-      options: {
-        config: '.jscsrc'
-      },
-      all: [
-        'Gruntfile.js',
-        'tasks/*.js',
-        '<%= nodeunit.tests %>'
-      ]
-    },
-
     // Before generating any new files, remove any previously-created files.
     clean: {
       tests: ['tmp']
@@ -81,7 +59,7 @@ module.exports = function (grunt) {
   // These plugins provide necessary tasks.
   require('load-grunt-tasks')(grunt, {scope: 'devDependencies'});
 
-  grunt.registerTask('lint', ['jshint', 'eslint', 'jscs']);
+  grunt.registerTask('lint', ['eslint']);
 
   // Whenever the "test" task is run, first clean the "tmp" dir, then run this
   // plugin's task(s), then test the result.

.jshintrc

@@ -106,7 +106,7 @@ grunt.initConfig({
 ```
 
 ## Contributing
-The project's coding style is laid out in the JSHint and JSCS configurations. Add unit tests for any new or changed functionality. Lint and test your code using [Grunt](http://gruntjs.com/).
+The project's coding style is laid out in the ESLint configuration. Add unit tests for any new or changed functionality. Lint and test your code using [Grunt](http://gruntjs.com/).
 
 ## License and copyright
 

Gruntfile.js

@@ -25,10 +25,8 @@
   "devDependencies": {
     "grunt": "^0.4.5",
     "grunt-contrib-clean": "^1.0.0",
-    "grunt-contrib-jshint": "^1.0.0",
     "grunt-contrib-nodeunit": "^1.0.0",
     "grunt-eslint": "^18.0.0",
-    "grunt-jscs": "^2.7.0",
     "load-grunt-tasks": "^3.1.0"
   },
   "peerDependencies": {