Fix lint of plugin code

Author: TrySound

.eslintignore

@@ -1,2 +1,3 @@
 test/fixtures
 test/expected
+createInlineWorker.js

index.js

@@ -1,56 +1,58 @@
-var WebWorkerTemplatePlugin = require("webpack/lib/webworker/WebWorkerTemplatePlugin");
-var SingleEntryPlugin = require("webpack/lib/SingleEntryPlugin");
-var path = require("path");
+const path = require('path');
+const WebWorkerTemplatePlugin = require('webpack/lib/webworker/WebWorkerTemplatePlugin');
+const SingleEntryPlugin = require('webpack/lib/SingleEntryPlugin');
+const loaderUtils = require('loader-utils');
 
-var loaderUtils = require("loader-utils");
-module.exports = function() {};
-module.exports.pitch = function(request) {
-  if(!this.webpack) throw new Error("Only usable with webpack");
+module.exports = function workerLoader() {};
+
+module.exports.pitch = function pitch(request) {
+  if (!this.webpack) throw new Error('Only usable with webpack');
   this.cacheable(false);
-  var callback = this.async();
-  var query = loaderUtils.parseQuery(this.query);
-  var filename = loaderUtils.interpolateName(this, query.name || "[hash].worker.js", {
+  const callback = this.async();
+  const query = loaderUtils.parseQuery(this.query);
+  const filename = loaderUtils.interpolateName(this, query.name || '[hash].worker.js', {
     context: query.context || this.options.context,
-    regExp: query.regExp
+    regExp: query.regExp,
   });
-  var outputOptions = {
-    filename: filename,
-    chunkFilename: "[id]." + filename,
-    namedChunkFilename: null
+  const outputOptions = {
+    filename,
+    chunkFilename: `[id].${filename}`,
+    namedChunkFilename: null,
   };
-  if(this.options && this.options.worker && this.options.worker.output) {
-    for(var name in this.options.worker.output) {
+  if (this.options && this.options.worker && this.options.worker.output) {
+    Object.keys(this.options.worker.output).forEach((name) => {
       outputOptions[name] = this.options.worker.output[name];
-    }
+    });
   }
-  var workerCompiler = this._compilation.createChildCompiler("worker", outputOptions);
+  const workerCompiler = this._compilation.createChildCompiler('worker', outputOptions);
   workerCompiler.apply(new WebWorkerTemplatePlugin(outputOptions));
-  workerCompiler.apply(new SingleEntryPlugin(this.context, "!!" + request, "main"));
-  if(this.options && this.options.worker && this.options.worker.plugins) {
-    this.options.worker.plugins.forEach(function(plugin) {
-      workerCompiler.apply(plugin);
-    });
+  workerCompiler.apply(new SingleEntryPlugin(this.context, `!!${request}`, 'main'));
+  if (this.options && this.options.worker && this.options.worker.plugins) {
+    this.options.worker.plugins.forEach(plugin => workerCompiler.apply(plugin));
   }
-  var subCache = "subcache " + __dirname + " " + request;
-  workerCompiler.plugin("compilation", function(compilation) {
-    if(compilation.cache) {
-      if(!compilation.cache[subCache])
+  const subCache = `subcache ${__dirname} ${request}`;
+  workerCompiler.plugin('compilation', (compilation) => {
+    if (compilation.cache) {
+      if (!compilation.cache[subCache]) {
         compilation.cache[subCache] = {};
+      }
       compilation.cache = compilation.cache[subCache];
     }
   });
-  workerCompiler.runAsChild(function(err, entries, compilation) {
-    if(err) return callback(err);
+  workerCompiler.runAsChild((err, entries, compilation) => {
+    if (err) return callback(err);
     if (entries[0]) {
-      var workerFile = entries[0].files[0];
-      var constructor = "new Worker(__webpack_public_path__ + " + JSON.stringify(workerFile) + ")";
-      if(query.inline) {
-        constructor = "require(" + JSON.stringify("!!" + path.join(__dirname, "createInlineWorker.js")) + ")(" +
-          JSON.stringify(compilation.assets[workerFile].source()) + ", __webpack_public_path__ + " + JSON.stringify(workerFile) + ")";
+      const [workerFile] = entries[0].files;
+      let constructor = `new Worker(__webpack_public_path__ + ${JSON.stringify(workerFile)})`;
+      if (query.inline) {
+        constructor = `require(${JSON.stringify(`!!${path.join(__dirname, 'createInlineWorker.js')}`)})(${
+          JSON.stringify(compilation.assets[workerFile].source())
+        }, __webpack_public_path__ + ${
+          JSON.stringify(workerFile)
+        })`;
       }
-      return callback(null, "module.exports = function() {\n\treturn " + constructor + ";\n};");
-    } else {
-      return callback(null, null);
+      return callback(null, `module.exports = function() {\n\treturn ${constructor};\n};`);
     }
+    return callback(null, null);
   });
 };

package.json

@@ -20,7 +20,9 @@
           "exports": "always-multiline",
           "functions": "never"
         }
-      ]
+      ],
+      "no-underscore-dangle": 0,
+      "no-param-reassign": 0
     }
   },
   "peerDependencies": {