Merge pull request #51 from jaebradley/linting

Add Linting

Author: jaebradley

.eslintrc.json

@@ -0,0 +1,10 @@
+{
+    "extends": "airbnb-base",
+    "plugins": [
+        "import",
+        "chai-expect"
+    ],
+    "rules": {
+      "class-methods-use-this": "off"
+    }
+}

.travis.yml

@@ -3,4 +3,4 @@ node_js:
 - 4.2.2
 sudo: false
 script:
-- npm run coveralls
+- npm run build

package.json

@@ -31,8 +31,13 @@
     "chai-as-promised": "^6.0.0",
     "chai-immutable": "^1.6.0",
     "coveralls": "^2.11.9",
+    "eslint": "^3.19.0",
+    "eslint-config-airbnb-base": "^11.2.0",
+    "eslint-plugin-chai-expect": "^1.1.1",
+    "eslint-plugin-import": "^2.3.0",
+    "eslint-plugin-mocha": "^4.10.1",
     "istanbul": "^1.0.0-alpha.2",
-    "mocha": "^2.4.5",
+    "mocha": "^2.5.3",
     "mocha-lcov-reporter": "^1.2.0 ",
     "sinon": "^2.1.0",
     "sinon-chai": "^2.9.0"
@@ -57,6 +62,8 @@
     "coverage": "babel-node ./node_modules/istanbul/lib/cli cover node_modules/mocha/bin/_mocha test --  --recursive --compilers js:babel-core/register",
     "coveralls": "npm run compile && npm run coverage && node_modules/.bin/coveralls < coverage/lcov.info",
     "prepublish": "npm run compile",
-    "test": "npm run compile && mocha --compilers js:babel-core/register"
+    "lint": "eslint src && eslint test",
+    "test": "npm run compile && mocha --recursive --compilers js:babel-core/register",
+    "build": "npm run lint && npm run coveralls"
   }
 }

src/Utilities.js

@@ -1,8 +1,6 @@
-'use es6';
+import { Record } from 'immutable';
 
-import {Record} from 'immutable';
-
-let defaults = {
+const defaults = {
   latitude: 0,
   longitude: 0,
 };

src/data/Coordinate.js

@@ -1,12 +1,10 @@
-'use es6';
-
 import { Record } from 'immutable';
 
 import DistanceUnit from './DistanceUnit';
 
 const defaults = {
   value: 0,
-  unit: DistanceUnit.MILE
+  unit: DistanceUnit.MILE,
 };
 
 export default class Distance extends Record(defaults) {}

src/data/Distance.js

@@ -1,5 +1,3 @@
-'use es6';
-
 import { Enum } from 'enumify';
 
 export default class DistanceUnit extends Enum {}

src/data/DistanceUnit.js

@@ -1,12 +1,10 @@
-'use es6';
-
 import { Record } from 'immutable';
 
 import TimeUnit from './TimeUnit';
 
 const defaults = {
   length: 0,
-  unit: TimeUnit.SECOND
+  unit: TimeUnit.SECOND,
 };
 
 export default class Duration extends Record(defaults) {

src/data/Duration.js

@@ -1,13 +1,11 @@
-'use es6';
-
-import {Record} from 'immutable';
+import { Record } from 'immutable';
 
 import Coordinate from './Coordinate';
 
-let defaults = {
+const defaults = {
   name: '',
   coordinate: new Coordinate(),
-}
+};
 
 export default class Location extends Record(defaults) {
 }

src/data/Location.js

@@ -1,12 +1,10 @@
-'use es6';
-
 import { Record } from 'immutable';
+
 import Duration from './Duration';
 
-let defaults = {
+const defaults = {
   productName: '',
   estimatedDuration: new Duration(),
 };
 
-export default class PickupTimeEstimate extends Record(defaults){
-};
+export default class PickupTimeEstimate extends Record(defaults) {}