Patrickshih/webpack.config-file

[shihpatrick]

Summary

• Rewrote build scripts as a webpack.config.js file
• Changed relevant build scripts to support new config file
• Utilized environmental variables as flags/parameters to separate and specify builds
  • env.mode = [development | production]
  • env.platform = [node | browser]
  • env.targ = [cjs | umd]
• upgraded webpack to 3.x, but could not do webpack 4.x because webpack 4.x does not support node4

With a webpack config file, I would be able to simplify the task to add typescript into the Javascript SDK. Also, the config file provides added modularity to the existing codebase.

Test plan

• Ran npm run build to ensure that dist/ files were being created
• Built all dist/ files twice (one with webpack.config.js file, one with master) and diff each of them
  • Expected: no difference between each file
  • Actual: Some noticeable differences. Discussed with @spencerwilson-optimizely, and the differences could be because of the upgrade from webpack 2.x.x to 4.x.x. Concluded that these differences should not affect the functional outcome.
  • Example of a diff-*.txt file (diff-cjs.txt):
    

[spencerwilson-optimizely]

Hooray, looks great!

[spencerwilson-optimizely]

nit: I'd probably like to rename the env option to --env.target rather than targ, since abbreviations can add unnecessary complexity.

[spencerwilson-optimizely]

I think these changes were obviated by #135. Going to close this.