use terser by default

vitejs · May 5, 2020 · 4fd2cdf · 4fd2cdf
1 parent b87ba7e
commit 4fd2cdf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/node/build.ts b/src/node/build.ts
@@ -158,10 +158,12 @@ export async function build(options: BuildOptions = {}): Promise<BuildResult> {
     resolver
   )
 
+  // terser is used by default for better compression, but the user can also
+  // opt-in to use esbuild which is orders of magnitude faster.
   const minifyPlugin = minify
-    ? minify === 'terser'
-      ? require('rollup-plugin-terser').terser()
-      : await createMinifyPlugin()
+    ? minify === 'esbuild'
+      ? await createMinifyPlugin()
+      : require('rollup-plugin-terser').terser()
     : null
 
   // lazy require rollup so that we don't load it when only using the dev server

diff --git a/src/node/esbuildService.ts b/src/node/esbuildService.ts
@@ -7,7 +7,6 @@ const transform = async (
   options: TransformOptions,
   operation: string
 ) => {
-  console.log(operation)
   try {
     const result = await service.transform(code, options)
     if (result.warnings.length) {