chore(lint): fix linter settings (#467)

This narrows down to a single eslint
configuration and fixes existing
linter issues. This also updates
to the latest webpack-contrib eslint
recommendations.

Fixes N/A

.eslintignore

@@ -1,3 +1,2 @@
-lib/*
 /node_modules
 /dist

.eslintrc.js

@@ -3,9 +3,16 @@ module.exports = {
   plugins: ['prettier'],
   extends: ['@webpack-contrib/eslint-config-webpack'],
   rules: {
+    "consistent-return": "off",
+    "no-console": "off",
+    "no-param-reassign": "off",
+    "no-underscore-dangle": "off",
+    "prefer-destructuring": ["error", {"object": false, "array": false}],
+    "prefer-rest-params": "off",
+    "strict": ["error", "safe"],
     'prettier/prettier': [
       'error',
       { singleQuote: true, trailingComma: 'es5', arrowParens: 'always' },
     ],
   },
-};
+};

lib/KarmaWebpackController.js

@@ -1,5 +1,3 @@
-/* eslint-disable no-console */
-
 const path = require('path');
 const fs = require('fs');
 const os = require('os');
@@ -21,12 +19,10 @@ class KarmaSyncPlugin {
       // read generated file content and store for karma preprocessor
       this.controller.bundlesContent = {};
       stats.toJson().assets.forEach((webpackFileObj) => {
-        const filePath = `${compiler.options.output.path}/${
-          webpackFileObj.name
-        }`;
+        const filePath = `${compiler.options.output.path}/${webpackFileObj.name}`;
         this.controller.bundlesContent[webpackFileObj.name] = fs.readFileSync(
           filePath,
-          'utf-8',
+          'utf-8'
         );
       });
 
@@ -88,7 +84,7 @@ class KarmaWebpackController {
       new KarmaSyncPlugin({
         karmaEmitter: emitter,
         controller: this,
-      }),
+      })
     );
   }
 
@@ -135,13 +131,13 @@ class KarmaWebpackController {
       if (this.webpackOptions.watch === true) {
         console.log('Webpack starts watching...');
         this.compiler = webpack(this.webpackOptions, (err, stats) =>
-          this.handleBuildResult(err, stats, resolve),
+          this.handleBuildResult(err, stats, resolve)
         );
 
         this.setupExitHandler(this.compiler);
       } else {
         this.compiler = webpack(this.webpackOptions).run((err, stats) =>
-          this.handleBuildResult(err, stats, resolve),
+          this.handleBuildResult(err, stats, resolve)
         );
       }
     });

package.json

@@ -44,7 +44,7 @@
     "@commitlint/cli": "^7.1.1",
     "@commitlint/config-conventional": "^7.1.1",
     "@webpack-contrib/defaults": "^2.5.0",
-    "@webpack-contrib/eslint-config-webpack": "^2.0.4",
+    "@webpack-contrib/eslint-config-webpack": "^3.0.0",
     "babel-cli": "^6.26.0",
     "babel-jest": "^23.4.2",
     "babel-plugin-transform-object-rest-spread": "^6.26.0",

test/integration/scenarios/basic-setup/basic-setup.test.js

@@ -29,7 +29,7 @@ describe('A basic karma-webpack setup', () => {
     plugins: [karmaWebpack, karmaChromeLauncher, karmaMocha, karmaChai],
     port: 2389,
     logLevel: 'ERROR',
-    singleRun: true,
+    singleRun: true
   };
 
   beforeAll((done) => {

test/integration/utils/ScenarioUtils.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 const karma = require('karma');
 
 const ScenarioUtils = { run };