ESLint class-methods-use-this

Author: roman

.eslintrc.js

@@ -37,7 +37,7 @@ module.exports = {
         "no-undef": 0,
         "no-unused-vars": 0,
         "no-empty-function": 0,
-        "class-methods-use-this": 0,
+        "class-methods-use-this": 1,
         "import/no-webpack-loader-syntax": 1,
         "import/newline-after-import": 1,
         "import/no-duplicates": 1,

README.md

@@ -28,6 +28,11 @@ npm i
 npm run start
 ```
 
+HUSKY
+---
+* Install: `npm i husky`
+* Uninstall: `npm r husky`
+
 TASKS
 ---
 | Task                        | Description                                                            |

package.json

@@ -21,8 +21,8 @@
     "tests:w": "npm run karma",
     "server": "webpack-dev-server --hot --progress --colors",
     "server:dashboard": "npm run server -- --env.dashboard",
-    "build": "webpack --progress --colors",
-    "build:prod": "npm run build -- --env.prod --display-modules",
+    "build": "webpack --progress --colors --display-modules",
+    "build:prod": "npm run build -- --env.prod",
     "dashboard": "webpack-dashboard -- npm run server:dashboard",
     "up": "node node_modules/updtr/bin/updtr --test-stdout --save-exact -R simple --test \"npm run test\"",
     "build:vendor-libs": "npm run build -- --env.vendorLibs --display-modules",
@@ -43,7 +43,8 @@
     "dev2b": "npm run p -q -- \"npm run tests:w\" \"npm run server\"",
     "stryker:run": "stryker run stryker.conf.js",
     "stryker:source": "npm run gulp -- stryker:source",
-    "stryker": "npm run stryker:source && npm run stryker:run"
+    "stryker": "npm run stryker:source && npm run stryker:run",
+    "prepush": "npm run t"
   },
   "engines": {
     "npm": ">=3"

src/app/app.errorhandler.ts

@@ -6,7 +6,7 @@ export class AppErrorHandler extends ErrorHandler {
         super(true);
     }
 
-    handleError(err: any) { // eslint-disable-line
-        console.error(err); // eslint-disable-line
+    handleError(err: any) { // eslint-disable-line class-methods-use-this
+        console.error(err); // eslint-disable-line no-console
     }
 }

src/main.ts

@@ -1,6 +1,4 @@
-if (process.env.NODE_ENV === 'production') {
-    require('core-js/es6');
-}
+require('core-js/es6');
 require('core-js/es7/reflect');
 require('core-js/es7/array');
 require('zone.js/dist/zone');
@@ -29,7 +27,7 @@ if (process.env.NODE_ENV === 'production') {
             private readonly appRef: ApplicationRef
         ) { }
 
-        hmrOnInit(store: any) {
+        hmrOnInit(store: any) { // eslint-disable-line class-methods-use-this
             // https://github.com/angularclass/angular2-hmr
         }
 
@@ -39,7 +37,7 @@ if (process.env.NODE_ENV === 'production') {
             removeNgStyles();
         }
 
-        hmrAfterDestroy(store: any) {
+        hmrAfterDestroy(store: any) { // eslint-disable-line class-methods-use-this
             store.disposeOldHosts();
             delete store.disposeOldHosts;
         }

webpack.config.ts

@@ -180,7 +180,7 @@ export = (options: Options = {}) => {
                     ]
                 },
                 {
-                    test: /app\.scss$/,
+                    test: /[\/\\]app\.scss$/,
                     use: (() => {
                         let result = [
                             { loader: 'css-loader' },