tasks: dry up paths to partial bundles

Author: etpinard

tasks/bundle.js

@@ -54,15 +54,11 @@ _bundle(constants.pathToPlotlyIndex, constants.pathToPlotlyDistWithMeta, {
 });
 
 // Browserify the plotly.js partial bundles
-constants.partialBundleNames.forEach(function(name) {
-    var pathToIndex = path.join(constants.pathToLib, 'index-' + name + '.js'),
-        pathToBundle = path.join(constants.pathToDist, 'plotly-' + name + '.js'),
-        pathToMinBundle = path.join(constants.pathToDist, 'plotly-' + name + '.min.js');
-
-    _bundle(pathToIndex, pathToBundle, {
+constants.partialBundlePaths.forEach(function(pathObj) {
+    _bundle(pathObj.index, pathObj.dist, {
         standalone: 'Plotly',
         debug: DEV,
-        pathToMinBundle: pathToMinBundle
+        pathToMinBundle: pathObj.distMin
     });
 });
 

tasks/util/constants.js

@@ -1,5 +1,4 @@
 var path = require('path');
-
 var pkg = require('../../package.json');
 
 var pathToRoot = path.join(__dirname, '../../');
@@ -13,6 +12,19 @@ var pathToTopojsonSrc = path.join(
     path.dirname(require.resolve('sane-topojson')), 'dist/'
 );
 
+var partialBundleNames = [
+    'basic', 'cartesian', 'geo', 'gl3d', 'gl2d', 'mapbox'
+];
+
+var partialBundlePaths = partialBundleNames.map(function(name) {
+    return {
+        name: name,
+        index: path.join(pathToLib, 'index-' + name + '.js'),
+        dist: path.join(pathToDist, 'plotly-' + name + '.js'),
+        distMin: path.join(pathToDist, 'plotly-' + name + '.min.js')
+    };
+});
+
 var year = (new Date()).getFullYear();
 
 module.exports = {
@@ -29,9 +41,8 @@ module.exports = {
     pathToPlotlyDistMin: path.join(pathToDist, 'plotly.min.js'),
     pathToPlotlyDistWithMeta: path.join(pathToDist, 'plotly-with-meta.js'),
 
-    partialBundleNames: [
-        'basic', 'cartesian', 'geo', 'gl3d', 'gl2d', 'mapbox'
-    ],
+    partialBundleNames: partialBundleNames,
+    partialBundlePaths: partialBundlePaths,
 
     pathToTopojsonSrc: pathToTopojsonSrc,
     pathToTopojsonDist: path.join(pathToDist, 'topojson/'),