From e4e95aa8a0aadb295ce85500829f05afca0f9d19 Mon Sep 17 00:00:00 2001
From: Maxime Beauchemin <maximebeauchemin@gmail.com>
Date: Tue, 10 Aug 2021 00:08:13 -0700
Subject: [PATCH] chore: remove TerserPlugin step for build

---
 Dockerfile                          | 2 +-
 superset-frontend/package-lock.json | 1 -
 superset-frontend/package.json      | 1 -
 superset-frontend/webpack.config.js | 9 ---------
 4 files changed, 1 insertion(+), 12 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index 6ca89e889fd7d..c73e6cad61988 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -64,7 +64,7 @@ RUN /frontend-mem-nag.sh \
 
 # Next, copy in the rest and let webpack do its thing
 COPY ./superset-frontend /app/superset-frontend
-# This is BY FAR the most expensive step (thanks Terser!)
+# This seems to be the most expensive step
 RUN cd /app/superset-frontend \
         && npm run ${BUILD_CMD} \
         && rm -rf node_modules
diff --git a/superset-frontend/package-lock.json b/superset-frontend/package-lock.json
index 51d88149d561c..e75cf48fd2297 100644
--- a/superset-frontend/package-lock.json
+++ b/superset-frontend/package-lock.json
@@ -247,7 +247,6 @@
         "storybook-addon-jsx": "^7.3.3",
         "storybook-addon-paddings": "^3.2.0",
         "style-loader": "^1.0.0",
-        "terser-webpack-plugin": "^1.1.0",
         "thread-loader": "^1.2.0",
         "transform-loader": "^0.2.3",
         "ts-jest": "^26.4.2",
diff --git a/superset-frontend/package.json b/superset-frontend/package.json
index a96ccedf7de95..b025107c6e11f 100644
--- a/superset-frontend/package.json
+++ b/superset-frontend/package.json
@@ -299,7 +299,6 @@
     "storybook-addon-jsx": "^7.3.3",
     "storybook-addon-paddings": "^3.2.0",
     "style-loader": "^1.0.0",
-    "terser-webpack-plugin": "^1.1.0",
     "thread-loader": "^1.2.0",
     "transform-loader": "^0.2.3",
     "ts-jest": "^26.4.2",
diff --git a/superset-frontend/webpack.config.js b/superset-frontend/webpack.config.js
index d84646cd100c2..327f6ac6277a7 100644
--- a/superset-frontend/webpack.config.js
+++ b/superset-frontend/webpack.config.js
@@ -27,7 +27,6 @@ const HtmlWebpackPlugin = require('html-webpack-plugin');
 const MiniCssExtractPlugin = require('mini-css-extract-plugin');
 const OptimizeCSSAssetsPlugin = require('optimize-css-assets-webpack-plugin');
 const SpeedMeasurePlugin = require('speed-measure-webpack-plugin');
-const TerserPlugin = require('terser-webpack-plugin');
 const ManifestPlugin = require('webpack-manifest-plugin');
 const ForkTsCheckerWebpackPlugin = require('fork-ts-checker-webpack-plugin');
 const parsedArgs = require('yargs').argv;
@@ -508,14 +507,6 @@ if (isDevMode) {
   if (hasSymlink) {
     console.log(''); // pure cosmetic new line
   }
-} else {
-  config.optimization.minimizer = [
-    new TerserPlugin({
-      cache: '.terser-plugin-cache/',
-      parallel: true,
-      extractComments: true,
-    }),
-  ];
 }
 
 // Bundle analyzer is disabled by default