Merge pull request #214 from devinmatte/js_linting

Linting Javascript

Author: devinmatte

.eslintrc.json

@@ -0,0 +1,25 @@
+{
+    "env": {
+        "browser": true,
+        "commonjs": true,
+        "es6": true
+    },
+    "extends": [
+        "standard"
+    ],
+    "globals": {
+        "Atomics": "readonly",
+        "SharedArrayBuffer": "readonly"
+    },
+    "parserOptions": {
+        "ecmaVersion": 2018
+    },
+    "rules": {
+        "no-undef": "off",
+        "no-unused-vars": "warn",
+        "no-prototype-builtins": "warn",
+        "no-redeclare": "warn",
+        "no-inner-declarations": "warn",
+        "no-throw-literal": "warn"
+    }
+}

.gitignore

@@ -8,6 +8,7 @@ img/schedules/*.png
 
 # Ignore node_modules dependencies
 node_modules/*
+package-lock.json
 
 # Ignore built files
 assets/prod/*

.travis.yml

@@ -5,11 +5,20 @@ php:
   - 7.2
   - 7.3
   - nightly
+node_js:
+  - 10
 
 matrix:
     allow_failures:
         - php: 7.0
+        - php: 7.2
+        - php: 7.3
         - php: nightly
 
+install:
+  - npm install
+
 script:
   - if find . -name "*.php" ! -path "./vendor/*" -exec php -l {} 2>&1 \; | grep "syntax error, unexpected"; then exit 1; fi
+  - npm run lint
+  - npm run build