feat(build): enable eslint on build

.eslintrc.yml

@@ -20,7 +20,7 @@ parserOptions:
     modules: true
 
 # # http://eslint.org/docs/rules/
-# rules:
+rules:
 #   # Possible Errors
 #   no-await-in-loop: off
 #   no-cond-assign: error
@@ -32,7 +32,7 @@ parserOptions:
 #   no-dupe-keys: error
 #   no-duplicate-case: error
 #   no-empty-character-class: error
-#   no-empty: error
+  no-empty: warn
 #   no-ex-assign: error
 #   no-extra-boolean-cast: error
 #   no-extra-parens: off

package.json

@@ -64,8 +64,6 @@
     "splitargs": "github:deviantony/splitargs#~0.2.0",
     "toastr": "github:CodeSeven/toastr#~2.1.3",
     "ui-select": "^0.19.8",
-    "webpack": "^4.20.2",
-    "webpack-cli": "^3.1.2",
     "xterm": "^3.1.0"
   },
   "devDependencies": {
@@ -98,6 +96,8 @@
     "load-grunt-tasks": "^3.5.2",
     "ngtemplate-loader": "^2.0.1",
     "style-loader": "^0.23.1",
-    "url-loader": "^1.1.1"
+    "url-loader": "^1.1.1",
+    "webpack": "^4.20.2",
+    "webpack-cli": "^3.1.2"
   }
 }

webpack.config.js

@@ -13,11 +13,11 @@ module.exports = {
         test: /\.js$/,
         exclude: /node_modules/,
         use: [
-          'auto-ngtemplate-loader'
-          // {
-          //   enforce: 'pre',
-          //   loader: 'eslint-loader'
-          // }
+          'auto-ngtemplate-loader',
+          {
+            // enforce: 'pre',
+            loader: 'eslint-loader'
+          }
         ]
       },
       {