feat: allow easily disabling reporters

unjs · Nov 4, 2018 · f96ad47 · f96ad47
1 parent 05cf301
commit f96ad47
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 18 deletions.
diff --git a/demo/webpack.config.js b/demo/webpack.config.js
@@ -29,6 +29,7 @@ const config = (name, color) => ({
       color,
       name,
       reporters: ['basic', 'fancy'],
+      basic: false,
       reporter: {
         update({ state }) {
           if (lastProgress !== state.progress && state.progress % 25 === 0) {

diff --git a/src/plugin.js b/src/plugin.js
@@ -60,35 +60,45 @@ export default class WebpackBarPlugin extends ProgressPlugin {
     }
 
     // Resolve reposters
-    this.reporters = this.reporters.filter(Boolean).map((_reporter) => {
-      let reporter = _reporter;
-      let reporterOptions = this.options[reporter] || {};
+    this.reporters = this.reporters
+      .filter(Boolean)
+      .map((_reporter) => {
+        if (this.options[_reporter] === false) {
+          return false;
+        }
+
+        let reporter = _reporter;
+        let reporterOptions = this.options[reporter] || {};
 
-      if (Array.isArray(_reporter)) {
+        if (Array.isArray(_reporter)) {
         reporter = _reporter[0]; // eslint-disable-line
-        if (_reporter[1]) {
+          if (_reporter[1] === false) {
+            return false;
+          }
+          if (_reporter[1]) {
           reporterOptions = _reporter[1]; // eslint-disable-line
+          }
         }
-      }
 
-      if (typeof reporter === 'string') {
-        if (reporters[reporter]) {
-          reporter = reporters[reporter];
-        } else {
+        if (typeof reporter === 'string') {
+          if (reporters[reporter]) {
+            reporter = reporters[reporter];
+          } else {
           reporter = require(reporter); // eslint-disable-line
+          }
         }
-      }
 
-      if (typeof reporter === 'function') {
-        if (typeof reporter.constructor === 'function') {
+        if (typeof reporter === 'function') {
+          if (typeof reporter.constructor === 'function') {
           reporter = new reporter(reporterOptions); // eslint-disable-line
-        } else {
-          reporter = reporter(reporterOptions);
+          } else {
+            reporter = reporter(reporterOptions);
+          }
         }
-      }
 
-      return reporter;
-    });
+        return reporter;
+      })
+      .filter(Boolean);
   }
 
   callReporters(fn, payload = {}) {