Remove Tapable.apply calls

webpack-contrib · Jan 23, 2018 · 8de49e4 · 8de49e4
1 parent 98816a1
commit 8de49e4
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/index.js b/src/index.js
@@ -52,16 +52,17 @@ export function pitch(request) {
   worker.compiler = this._compilation
     .createChildCompiler('worker', worker.options);
 
-  worker.compiler.apply(new WebWorkerTemplatePlugin(worker.options));
+  new WebWorkerTemplatePlugin(worker.options).apply(worker.compiler);
 
   if (this.target !== 'webworker' && this.target !== 'web') {
-    worker.compiler.apply(new NodeTargetPlugin());
+    new NodeTargetPlugin().apply(worker.compiler);
   }
 
-  worker.compiler.apply(new SingleEntryPlugin(this.context, `!!${request}`, 'main'));
+  new SingleEntryPlugin(this.context, `!!${request}`, 'main').apply(worker.compiler);
 
   const subCache = `subcache ${__dirname} ${request}`;
 
+  // TODO use `worker.compiler.hooks.compilation.tap()` instead
   worker.compiler.plugin('compilation', (compilation) => {
     if (compilation.cache) {
       if (!compilation.cache[subCache]) compilation.cache[subCache] = {};