[babel 8] Remove uglify target support in preset-env (#12594)

nicolo-ribaudo · JLHwung · web-flow · commit 606c91cd3317 · 2021-01-08T17:58:44.000+01:00
Co-authored-by: Huáng Jùnliàng &lt;jlhwung@gmail.com&gt;
diff --git a/packages/babel-preset-env/src/index.js b/packages/babel-preset-env/src/index.js
@@ -242,17 +242,20 @@ export default declare((api, opts) => {
     corejs: { version: corejs, proposals },
     browserslistEnv,
   } = normalizeOptions(opts);
-  // TODO: remove this in next major
-  let hasUglifyTarget = false;
 
-  if (optionsTargets?.uglify) {
-    hasUglifyTarget = true;
-    delete optionsTargets.uglify;
+  if (!process.env.BABEL_8_BREAKING) {
+    // eslint-disable-next-line no-var
+    var hasUglifyTarget = false;
 
-    console.log("");
-    console.log("The uglify target has been deprecated. Set the top level");
-    console.log("option `forceAllTransforms: true` instead.");
-    console.log("");
+    if (optionsTargets?.uglify) {
+      hasUglifyTarget = true;
+      delete optionsTargets.uglify;
+
+      console.log("");
+      console.log("The uglify target has been deprecated. Set the top level");
+      console.log("option `forceAllTransforms: true` instead.");
+      console.log("");
+    }
   }
 
   if (optionsTargets?.esmodules && optionsTargets.browsers) {
@@ -275,7 +278,13 @@ export default declare((api, opts) => {
   const include = transformIncludesAndExcludes(optionsInclude);
   const exclude = transformIncludesAndExcludes(optionsExclude);
 
-  const transformTargets = forceAllTransforms || hasUglifyTarget ? {} : targets;
+  const transformTargets = (
+    process.env.BABEL_8_BREAKING
+      ? forceAllTransforms
+      : forceAllTransforms || hasUglifyTarget
+  )
+    ? {}
+    : targets;
 
   const compatData = getPluginList(shippedProposals, bugfixes);
   const shouldSkipExportNamespaceFrom =
diff --git a/packages/babel-preset-env/test/fixtures/debug/entry-no-corejs-uglify/options.json b/packages/babel-preset-env/test/fixtures/debug/entry-no-corejs-uglify/options.json
@@ -1,4 +1,5 @@
 {
+  "BABEL_8_BREAKING": false,
   "validateLogs": true,
   "ignoreOutput": true,
   "presets": [

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`{`
	`2`	`+ "BABEL_8_BREAKING": false,`
`2`	`3`	`"validateLogs": true,`
`3`	`4`	`"ignoreOutput": true,`
`4`	`5`	`"presets": [`