REVIEW: Add strip-loader to remove console.log

apache · Dec 28, 2016 · 920589b · 920589b
1 parent 6aa25b8
commit 920589b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/zeppelin-web/package.json b/zeppelin-web/package.json
@@ -66,6 +66,7 @@
     "npm-run-all": "^3.1.2",
     "postcss-loader": "^1.2.1",
     "raw-loader": "^0.5.1",
+    "strip-loader": "^0.1.2",
     "style-loader": "^0.13.1",
     "time-grunt": "^0.3.1",
     "webpack": "^1.14.0",

diff --git a/zeppelin-web/webpack.config.js b/zeppelin-web/webpack.config.js
@@ -130,6 +130,11 @@ module.exports = function makeWebpackConfig () {
    * This handles most of the magic responsible for converting modules
    */
 
+  var jsLoaders = ['ng-annotate', 'babel-loader'];
+  if (isProd) {
+    jsLoaders.push('strip-loader?strip[]=console.log');
+  }
+
   // Initialize module
   config.module = {
     preLoaders: [],
@@ -139,7 +144,7 @@ module.exports = function makeWebpackConfig () {
       // Transpile .js files using babel-loader
       // Compiles ES6 and ES7 into ES5 code
       test: /\.js$/,
-      loaders: ['ng-annotate', 'babel-loader'],
+      loaders: jsLoaders,
       exclude: /(node_modules|bower_components)/,
     }, {
       // CSS LOADER