fix(tooling): webpack-prod-build-for-npm (#383)

.circleci/config.yml

@@ -61,7 +61,7 @@ jobs:
       - checkout_from_workspace
       - run:
           name: Build
-          command: yarn run build && yarn workspace @webex/widgets run build:src
+          command: yarn run build:prod && yarn workspace @webex/widgets run build:src
       - persist_to_workspace:
           root: .
           paths:

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,49 @@
+<!--
+Hey there,\
+Thank you for taking the time to improve our code! 🙂\
+Please let us know why this change is necessary and what testing you have performed. \
+This ensures our reviewers understand the impact of your change. \
+
+**IMPORTANT**
+FAILING TO FILL OUT THIS TEMPLATE WILL RESULT IN REJECTION OF YOUR PULL REQUEST
+This is for compliance purposes with FedRAMP program.
+-->
+
+# COMPLETES #< INSERT LINK TO ISSUE >
+
+## This pull request addresses
+
+< DESCRIBE THE CONTEXT OF THE ISSUE >
+
+## by making the following changes
+
+< DESCRIBE YOUR CHANGES >
+
+<!-- You may include screenshots -->
+
+### Change Type
+
+<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: -->
+
+- [ ] Bug fix (non-breaking change which fixes an issue)
+- [ ] New feature (non-breaking change which adds functionality)
+- [ ] Breaking change (fix or feature that would cause existing functionality to change)
+- [ ] Documentation update
+- [ ] Tooling change
+- [ ] Internal code refactor
+
+## The following scenarios were tested
+
+- [ ] The testing is done with the amplify link
+< ENUMERATE TESTS PERFORMED, WHETHER MANUAL OR AUTOMATED >
+
+### Checklist before merging
+
+- [ ] I have not skipped any automated checks
+- [ ] All existing and new tests passed
+- [ ] I have updated the testing document
+- [ ] I have tested the functionality with amplify link
+
+---
+
+Make sure to have followed the [contributing guidelines](/~https://github.com/webex/webex-js-sdk/blob/master/CONTRIBUTING.md#submitting-a-pull-request) before submitting.

package.json

@@ -42,7 +42,9 @@
     "clean:dist": "yarn workspaces foreach --all --topological --parallel run clean:dist",
     "test:unit": "yarn run test:tooling && yarn run test:cc-widgets",
     "test:tooling": "jest --coverage",
-    "test:cc-widgets": "yarn workspaces foreach --all --parallel --exclude webex-widgets --exclude samples-cc-wc-app --exclude samples-cc-react-app run test:unit",
+    "test:cc-widgets": "yarn workspaces foreach --all --exclude webex-widgets --exclude samples-cc-wc-app --exclude samples-cc-react-app run test:unit",
+    "build:dev": "NODE_ENV=development yarn build",
+    "build:prod": "NODE_ENV=production  yarn build",
     "build": "yarn workspaces foreach --all --topological --parallel --exclude samples-cc-react-app --exclude samples-cc-wc-app --exclude samples-meeting-app run build:src",
     "samples:build": "yarn workspaces foreach --all --parallel --include samples-cc-react-app --include samples-cc-wc-app --include samples-meeting-app run build:src && ./copy_to_docs.sh",
     "samples:serve": "http-server docs",

webpack.config.js

@@ -2,7 +2,7 @@ const webpack = require('webpack');
 const path = require('path');
 
 module.exports = {
-  mode: 'development',
+  mode: process.env.NODE_ENV || 'development',
   entry: './src/index.ts',
   resolve: {
     extensions: ['.ts', '.tsx', '.js', '.jsx', '.scss'],