Merge pull request #57 from ember-cli/add-linting

Add linting via ESLint

.eslintrc.js

@@ -0,0 +1,18 @@
+module.exports = {
+  env: {
+    node: true,
+  },
+  plugins: ['node'], 
+  extends: ['eslint:recommended', 'plugin:node/recommended'], 
+  parserOptions: {
+    ecmaVersion: 2017
+  },
+  overrides: [
+    {
+      files: ['tests/**/*-test.js'],
+      env: {
+        qunit: true,
+      }
+    }
+  ],
+};

.travis.yml

@@ -6,4 +6,5 @@ node_js:
   - 10
 
 script:
+  - yarn lint:js
   - yarn test

index.js

@@ -63,8 +63,8 @@ module.exports = {
     process.env.EMBER_CLI_INJECT_LIVE_RELOAD_PORT = options.liveReloadPort;
     process.env.EMBER_CLI_INJECT_LIVE_RELOAD_BASEURL = baseURL;
 
-    let baseURLWithoutHost = baseURL.replace(/^https?:\/\/[^\/]+/, '');
-    app.use(baseURLWithoutHost + 'ember-cli-live-reload.js', function(request, response, next) {
+    let baseURLWithoutHost = baseURL.replace(/^https?:\/\/[^/]+/, '');
+    app.use(baseURLWithoutHost + 'ember-cli-live-reload.js', function(request, response) {
       response.contentType('text/javascript');
       response.send(self.dynamicScript(options));
     });

package.json

@@ -15,16 +15,22 @@
   },
   "license": "MIT",
   "author": "Robert Jackson",
+  "files": [
+    "index.js"
+  ],
   "main": "index.js",
   "scripts": {
     "changelog": "lerna-changelog",
+    "lint:js": "eslint .",
     "test": "qunit tests"
   },
   "dependencies": {
     "clean-base-url": "^1.0.0",
     "ember-cli-version-checker": "^2.1.2"
   },
   "devDependencies": {
+    "eslint": "^4.0.0",
+    "eslint-plugin-node": "^6",
     "express": "^4.16.3",
     "lerna-changelog": "^0.7.0",
     "qunit": "~2.6.1"