Merge pull request #3129 from salesforce-ux/chore/gulp-dist

Refactor dist.js to use Gulp

gulpfile.js

@@ -7,10 +7,10 @@ import glob from 'glob';
 import gulp from 'gulp';
 import path from 'path';
 import touch from 'touch';
-import webpack from 'webpack';
 import { argv } from 'yargs';
 
 import * as accessibility from './scripts/gulp/accessibility';
+import * as dist from './scripts/gulp/dist';
 import * as examples from './scripts/gulp/generate/examples';
 import * as tokens from './scripts/gulp/generate/tokens';
 import * as lint from './scripts/gulp/lint';
@@ -183,6 +183,42 @@ export const watch = () =>
     });
   });
 
+// /////////////////////////////////////////////////////////
+// Dist
+// /////////////////////////////////////////////////////////
+
+gulp.task(
+  'dist',
+  gulp.series(
+    withName('dist:clean:before')(dist.cleanBefore),
+    gulp.parallel(
+      withName('dist:copyRoot')(dist.copyRoot),
+      withName('dist:copySass')(dist.copySass),
+      withName('dist:copySassLicense')(dist.copySassLicense),
+      withName('dist:copyIcons')(dist.copyIcons),
+      withName('dist:copyIconsMeta')(dist.copyIconsMeta),
+      withName('dist:copyFonts')(dist.copyFonts),
+      withName('dist:copyFontsLicense')(dist.copyFontsLicense),
+      withName('dist:copyImages')(dist.copyImages),
+      withName('dist:copyImagesLicense')(dist.copyImagesLicense),
+      withName('dist:copySwatches')(dist.copySwatches),
+      withName('dist:copyDesignTokens')(dist.copyDesignTokens),
+      withName('dist:copyComponentDesignTokens')(dist.copyComponentDesignTokens)
+    ),
+    withName('dist:sass')(dist.sass),
+    withName('dist:minifyCss')(dist.minifyCss),
+    gulp.parallel(
+      withName('dist:versionBlock')(dist.versionBlock),
+      withName('dist:versionInline')(dist.versionInline),
+      withName('dist:buildInfo')(dist.buildInfo),
+      withName('dist:writeUI')(dist.writeUI),
+      withName('dist:writeLibrary')(dist.writeLibrary),
+      withName('dist:packageJson')(dist.packageJson)
+    ),
+    withName('dist:clean:after')(dist.cleanAfter)
+  )
+);
+
 // /////////////////////////////////////////////////////////
 // Travis
 // /////////////////////////////////////////////////////////
@@ -196,6 +232,7 @@ gulp.task('travis', done => {
     'generate:examples:wrapped',
     withName('travis:snapshots')(travis.createSnapshots),
     withName('travis:lint:examples')(travis.lintExamples),
+    'dist',
     withName('travis:publish')(travis.publishBuild)
   )(done);
 });

package.json

@@ -146,6 +146,7 @@
     "stylestats": "7.0.1",
     "temp": "0.8.3",
     "theo": "6.0.0-beta.2",
+    "thread-loader": "1.1.5",
     "through2": "2.0.3",
     "tinycolor2": "1.4.1",
     "touch": "3.1.0",

scripts/compile/__tests__/__snapshots__/bundle.spec.js.snap

@@ -12,6 +12,11 @@ exports[`doesn't change configs 1`] = `
           \\"test\\": {
           },
           \\"use\\": [
+            {
+              \\"loader\\": \\"thread-loader\\",
+              \\"options\\": {
+              }
+            },
             {
               \\"loader\\": \\"babel-loader\\",
               \\"options\\": {
@@ -48,6 +53,11 @@ exports[`doesn't change configs 1`] = `
           \\"test\\": {
           },
           \\"use\\": [
+            {
+              \\"loader\\": \\"thread-loader\\",
+              \\"options\\": {
+              }
+            },
             {
               \\"loader\\": \\"babel-loader\\",
               \\"options\\": {
@@ -103,6 +113,11 @@ exports[`doesn't change configs 1`] = `
           \\"test\\": {
           },
           \\"use\\": [
+            {
+              \\"loader\\": \\"thread-loader\\",
+              \\"options\\": {
+              }
+            },
             {
               \\"loader\\": \\"babel-loader\\",
               \\"options\\": {
@@ -139,6 +154,11 @@ exports[`doesn't change configs 1`] = `
           \\"test\\": {
           },
           \\"use\\": [
+            {
+              \\"loader\\": \\"thread-loader\\",
+              \\"options\\": {
+              }
+            },
             {
               \\"loader\\": \\"babel-loader\\",
               \\"options\\": {
@@ -202,6 +222,11 @@ exports[`doesn't change configs 1`] = `
           \\"test\\": {
           },
           \\"use\\": [
+            {
+              \\"loader\\": \\"thread-loader\\",
+              \\"options\\": {
+              }
+            },
             {
               \\"loader\\": \\"babel-loader\\",
               \\"options\\": {
@@ -238,6 +263,11 @@ exports[`doesn't change configs 1`] = `
           \\"test\\": {
           },
           \\"use\\": [
+            {
+              \\"loader\\": \\"thread-loader\\",
+              \\"options\\": {
+              }
+            },
             {
               \\"loader\\": \\"babel-loader\\",
               \\"options\\": {

scripts/compile/bundle.js

@@ -109,12 +109,13 @@ const compile = configs =>
 
 const compileLibrary = () => configs.chain(compile);
 
-const writeManifest = () =>
+const writeManifest = stats =>
   manifest
     .map(m => JSON.stringify(m, null, 2))
     .chain(contents =>
       writeFile(path.join(paths.dist, 'manifest.json'), contents)
-    );
+    )
+    .map(() => stats);
 
 // createLibrary :: () -> Task Error (List Stats)
 const createLibrary = () => compileLibrary().chain(writeManifest);

scripts/compile/webpack.config.js

@@ -15,6 +15,10 @@ module.exports = I.fromJS({
         test: /\.jsx?$/,
         exclude: /node_modules/,
         use: [
+          {
+            loader: 'thread-loader',
+            options: {}
+          },
           {
             loader: 'babel-loader',
             options: {
@@ -39,6 +43,10 @@ module.exports = I.fromJS({
         test: /\.mdx$/,
         exclude: /node_modules/,
         use: [
+          {
+            loader: 'thread-loader',
+            options: {}
+          },
           {
             loader: 'babel-loader',
             options: {