Merge pull request #6150 from plotly/revise-extra-bundles

archmoj · web-flow · commit 13a37733b0c5 · 2022-03-18T15:35:06.000-04:00
Revise extra bundles creation
diff --git a/package.json b/package.json
@@ -32,7 +32,7 @@
     "use-draftlogs": "node tasks/use_draftlogs.js",
     "empty-draftlogs": "node tasks/empty_draftlogs.js",
     "empty-dist": "node tasks/empty_dist.js",
-    "build": "npm run empty-dist && npm run preprocess && npm run find-strings && npm run bundle && npm run extra-bundles basic && npm run extra-bundles cartesian && npm run extra-bundles geo && npm run extra-bundles gl2d && npm run extra-bundles gl3d && npm run extra-bundles mapbox && npm run extra-bundles finance && npm run locales && npm run schema dist && npm run stats",
+    "build": "npm run empty-dist && npm run preprocess && npm run find-strings && npm run bundle && npm run extra-bundles && npm run locales && npm run schema dist && npm run stats",
     "regl-codegen": "node devtools/regl_codegen/server.js",
     "cibuild": "npm run empty-dist && npm run preprocess && node tasks/cibundle.js",
     "watch": "node tasks/watch.js",
diff --git a/tasks/bundle.js b/tasks/bundle.js
@@ -50,8 +50,8 @@ tasks.push(function(done) {
         standalone: 'Plotly',
         pathToMinBundle: pathToPlotlyStrictDistMin
     }, function() {
-        prependFile.sync(pathToPlotlyStrictDist, header, common.throwOnError);
-        prependFile.sync(pathToPlotlyStrictDistMin, header, common.throwOnError);
+        prependFile.sync(pathToPlotlyStrictDist, header.replace('plotly.js', 'plotly.js (strict)'), common.throwOnError);
+        prependFile.sync(pathToPlotlyStrictDistMin, header.replace('plotly.js', 'plotly.js (strict - minified)'), common.throwOnError);
 
         done();
     });
diff --git a/tasks/extra_bundles.js b/tasks/extra_bundles.js
@@ -30,6 +30,9 @@ var tasks = [];
 for(var i = 0; i < list.length; i++) {
     var opts = list[i];
 
+    // strict bundle is no longer a partial bundle and generated with bundles
+    if(opts.name === 'strict') continue;
+
     partialBundle(tasks, {
         name: opts.name,
         index: opts.index,
