refactor: use function factory instead of class instantiation

mjeanroy · Aug 8, 2019 · 98888e0 · 98888e0
1 parent e549ed8
commit 98888e0
Show file tree

Hide file tree

Showing 4 changed files with 67 additions and 55 deletions.
diff --git a/src/index-rollup-legacy.js b/src/index-rollup-legacy.js
@@ -25,10 +25,10 @@
 'use strict';
 
 const _ = require('lodash');
-const LicensePlugin = require('./license-plugin.js');
+const licensePlugin = require('./license-plugin.js');
 
 module.exports = (options = {}) => {
-  const plugin = new LicensePlugin(options);
+  const plugin = licensePlugin(options);
 
   return {
     /**

diff --git a/src/index-rollup-stable.js b/src/index-rollup-stable.js
@@ -25,10 +25,10 @@
 'use strict';
 
 const _ = require('lodash');
-const LicensePlugin = require('./license-plugin.js');
+const licensePlugin = require('./license-plugin.js');
 
 module.exports = (options = {}) => {
-  const plugin = new LicensePlugin(options);
+  const plugin = licensePlugin(options);
 
   return {
     /**

diff --git a/src/license-plugin.js b/src/license-plugin.js
@@ -147,8 +147,9 @@ function fixSourceMapOptions(options) {
 
 /**
  * Rollup Plugin.
+ * @class
  */
-module.exports = class LicensePlugin {
+class LicensePlugin {
   /**
    * Initialize plugin.
    *
@@ -475,4 +476,15 @@ module.exports = class LicensePlugin {
 
     return COMMENT_STYLES[style] ? generateBlockComment(text, COMMENT_STYLES[style]) : text;
   }
+}
+
+/**
+ * Create new `rollup-plugin-license` instance with given
+ * options.
+ *
+ * @param {Object} options Option object.
+ * @return {LicensePlugin} The new instance.
+ */
+module.exports = function licensePlugin(options) {
+  return new LicensePlugin(options);
 };