Merge pull request #271 from rackerlabs/refactor-eslint

chore(linting): reconfigure eslint

Author: CITguy

.eslintrc.json

@@ -1,63 +1,38 @@
 {
-    "env": {
-        "browser": true,
-        "es6": true
-    },
-    "extends": "eslint:recommended",
-    "parserOptions": {
-        "sourceType": "module"
-    },
-    "globals": {
-        "$": false,
-        "module": false,
-        "require": false,
-        "ShadyCSS": false,
-        "Vue": false
-    },
+    "extends": "./.eslintrc.prod.json",
     "rules": {
-        "array-bracket-newline": [2, { "minItems": 3 }],
-        "array-bracket-spacing": [2, "always"],
-        "array-element-newline": [2, { 
-          "minItems": 3,
-          "multiline": true
-        }],
-        "brace-style": [2, "1tbs", {
-            "allowSingleLine": false
-        }],
-        "camelcase": 2,
-        "comma-dangle": [2, "always-multiline"],
-        "comma-style": [2, "last"],
-        "complexity": [2, 10],
-        "curly": [2, "all"],
-        "eol-last": 2,
-        "eqeqeq": 2,
-        "indent": [2, 4, { "SwitchCase": 1 }],
-        "keyword-spacing": 2,
-        "max-depth": [2, 3],
-        "max-len": [2, 120, { "ignoreUrls": true }],
-        "new-cap": 2,
-        "no-bitwise": 2,
-        "no-caller": 2,
-        "no-empty": 2,
-        "no-multiple-empty-lines": [2, {
-            "max": 1,
-            "maxBOF": 0
-        }],
-        "no-undef": 2,
-        "no-unused-expressions": [2, {
-            "allowTernary": true,
-            "allowShortCircuit": true
-        }],
-        "no-unused-vars": 2,
-        "no-use-before-define": [2, { "functions": false }],
-        "object-curly-spacing": [2, "always"],
-        "quotes": [2, "single"],
-        "semi": [2, "always"],
-        "space-before-function-paren": [2, "always"],
-        "space-in-parens": [2, "never"],
-        "space-infix-ops": 2,
-        "space-unary-ops": 2,
-        "strict": 0,
-        "wrap-iife": [2, "inside"]
+        "array-bracket-newline": 1,
+        "array-bracket-spacing": 1,
+        "array-element-newline": 1,
+        "brace-style": 1,
+        "camelcase": 1,
+        "comma-dangle": 1,
+        "comma-style": 1,
+        "complexity": 1,
+        "curly": 1,
+        "eol-last": 1,
+        "eqeqeq": 1,
+        "indent": 1,
+        "keyword-spacing": 1,
+        "max-depth": 1,
+        "max-len": 1,
+        "new-cap": 1,
+        "no-bitwise": 1,
+        "no-caller": 1,
+        "no-console": 1,
+        "no-empty": 1,
+        "no-multiple-empty-lines": 1,
+        "no-undef": 1,
+        "no-unused-expressions": 1,
+        "no-unused-vars": 1,
+        "no-use-before-define": 1,
+        "object-curly-spacing": 1,
+        "quotes": 1,
+        "semi": 1,
+        "space-before-function-paren": 1,
+        "space-in-parens": 1,
+        "space-infix-ops": 1,
+        "space-unary-ops": 1,
+        "wrap-iife": 1
     }
 }

.eslintrc.prod.json

@@ -0,0 +1,64 @@
+{
+    "env": {
+        "browser": true,
+        "es6": true
+    },
+    "extends": "eslint:recommended",
+    "parserOptions": {
+        "sourceType": "module"
+    },
+    "globals": {
+        "$": false,
+        "module": false,
+        "require": false,
+        "ShadyCSS": false,
+        "Vue": false
+    },
+    "rules": {
+        "array-bracket-newline": [2, { "minItems": 3 }],
+        "array-bracket-spacing": [2, "always"],
+        "array-element-newline": [2, {
+          "minItems": 3,
+          "multiline": true
+        }],
+        "brace-style": [2, "1tbs", {
+            "allowSingleLine": false
+        }],
+        "camelcase": 2,
+        "comma-dangle": [2, "always-multiline"],
+        "comma-style": [2, "last"],
+        "complexity": [2, 10],
+        "curly": [2, "all"],
+        "eol-last": 2,
+        "eqeqeq": 2,
+        "indent": [2, 4, { "SwitchCase": 1 }],
+        "keyword-spacing": 2,
+        "max-len": [2, 120, { "ignoreUrls": true }],
+        "new-cap": 2,
+        "no-bitwise": 2,
+        "no-caller": 2,
+        "no-empty": 2,
+        "no-multiple-empty-lines": [2, {
+            "max": 1,
+            "maxBOF": 0
+        }],
+        "no-undef": 2,
+        "no-unused-expressions": [2, {
+            "allowTernary": true,
+            "allowShortCircuit": true
+        }],
+        "no-unused-vars": [2, {
+            "argsIgnorePattern": "^_"
+        }],
+        "no-use-before-define": [2, { "functions": false }],
+        "object-curly-spacing": [2, "always"],
+        "quotes": [2, "single"],
+        "semi": [2, "always"],
+        "space-before-function-paren": [2, "always"],
+        "space-in-parens": [2, "never"],
+        "space-infix-ops": 2,
+        "space-unary-ops": 2,
+        "strict": 0,
+        "wrap-iife": [2, "inside"]
+    }
+}

.travis.yml

@@ -22,4 +22,5 @@ before_script:
 - yarn start > /dev/null 2>&1 &
 - while ! curl --silent http://localhost:4444/wd/hub > /dev/null 2>&1; do sleep 1; done
 script:
+- yarn lint
 - yarn test

package.json

@@ -62,7 +62,7 @@
     "generate": "bin/generate.js",
     "preghpages": "yarn generate",
     "ghpages": "bin/ghpages.js",
-    "lint": "eslint -c .eslintrc.json src",
+    "lint": "eslint -c .eslintrc.prod.json src",
     "postversion": "git push origin --tags",
     "prepublishOnly": "yarn compile",
     "rollup": "rollup -c rollup.config.js",