Add Linting (#293)

* Add linting

* Make specs pass linter

* Make lib/helpers pass linter

* Make lib/components pass linter

This also does some signfiicant refactoring of the code
to appease the linter's pro-ES2015+ stance.

closes #289
closes #286

* Make travis run the lint task as well as specs

closes #284

Author: claydiffrient

Diff for: .eslintrc.js

@@ -0,0 +1,25 @@
+module.exports = {
+  "env": {
+    "es6": true,
+    "browser": true
+  },
+  "extends": "airbnb",
+  "parserOptions": {
+    "ecmaVersion": 7,
+    "ecmaFeatures": {
+      "experimentalObjectRestSpread": true,
+      "jsx": true
+    },
+    "sourceType": "module"
+  },
+  "parser": "babel-eslint",
+  rules: {
+    "comma-dangle": [2, "only-multiline"],
+    "max-len": [1, {"code": 140}],
+    "no-continue": [0],
+    "no-plusplus": [0],
+    "space-before-function-paren": [2, "always"],
+    "import/no-extraneous-dependencies": [2, {"devDependencies": true}],
+    "react/jsx-filename-extension": ["error", {"extensions": [".js"]}]
+  },
+};

Diff for: Makefile

@@ -17,6 +17,7 @@ help: info
 	@echo "  make deps         - install all dependencies."
 	@echo "  make serve        - start the server."
 	@echo "  make tests        - run tests."
+	@echo "  make lint         - run lint."
 	@echo "  make docs         - build and serve the docs."
 	@echo "  make build        - build project artifacts."
 	@echo "  make publish      - build and publish version on npm."
@@ -48,6 +49,9 @@ tests:
 tests-ci:
 	@npm run test -- --single-run
 
+lint:
+	@npm run lint
+
 docs: build-docs
 	gitbook serve
 

Diff for: eslint.json

@@ -14,20 +14,28 @@
   },
   "scripts": {
     "start": "./node_modules/.bin/webpack-dev-server --inline --host 127.0.0.1 --content-base examples/",
-    "test": "cross-env NODE_ENV=test karma start"
+    "test": "cross-env NODE_ENV=test karma start",
+    "test:full": "npm-run-all -p 'test -- --single-run' lint",
+    "lint": "eslint lib/ specs/"
   },
   "authors": [
     "Ryan Florence"
   ],
   "license": "MIT",
   "devDependencies": {
     "babel-core": "^6.7.4",
+    "babel-eslint": "^7.1.1",
     "babel-loader": "^6.2.4",
     "babel-preset-es2015": "^6.6.0",
     "babel-preset-react": "^6.5.0",
     "babel-preset-stage-2": "^6.24.1",
     "cross-env": "^5.0.1",
     "envify": "^3.4.1",
+    "eslint": "^3.9.1",
+    "eslint-config-airbnb": "latest",
+    "eslint-plugin-import": "^2.1.0",
+    "eslint-plugin-jsx-a11y": "^2.2.3",
+    "eslint-plugin-react": "^6.6.0",
     "expect": "^1.20.2",
     "gitbook-cli": "^2.3.0",
     "karma": "^1.3.0",

Diff for: package.json

@@ -1,10 +1,9 @@
 const testsContext = require.context('.', true, /spec$/);
-testsContext.keys().forEach(function(path) {
-    try {
-        testsContext(path);
-    } catch(err) {
-      debugger
-      console.error(`[ERROR] WITH SPEC FILE: ${path}`);
-      console.error(err);
-    }
+testsContext.keys().forEach((path) => {
+  try {
+    testsContext(path);
+  } catch (err) {
+    console.error(`[ERROR] WITH SPEC FILE: ${path}`);
+    console.error(err);
+  }
 });

Diff for: specs/spec_index.js

@@ -1,7 +1,8 @@
-var commonConfig = require('./webpack.config');
+const commonConfig = require('./webpack.config');
 
 commonConfig.plugins = [];
 commonConfig.entry = undefined;
 commonConfig.debug = true;
+commonConfig.devtool = 'inline-source-map';
 
 module.exports = commonConfig;