fix: warn about dynamic import polyfill only during build (#3446)

patak-dev · web-flow · commit 5fe05507dd28 · 2021-05-17T22:10:44.000+08:00
diff --git a/packages/vite/src/node/plugins/dynamicImportPolyfill.ts b/packages/vite/src/node/plugins/dynamicImportPolyfill.ts
@@ -31,7 +31,7 @@ export function dynamicImportPolyfillPlugin(config: ResolvedConfig): Plugin {
     },
     load(id) {
       if (id === polyfillId) {
-        if (!enabled) {
+        if (!enabled && config.command === 'build') {
           config.logger.warnOnce(
             `\n'vite/dynamic-import-polyfill' is no longer needed if you target modern browsers`
           )

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ export function dynamicImportPolyfillPlugin(config: ResolvedConfig): Plugin {`
`31`	`31`	`},`
`32`	`32`	`load(id) {`
`33`	`33`	`if (id === polyfillId) {`
`34`		`- if (!enabled) {`
	`34`	`+ if (!enabled && config.command === 'build') {`
`35`	`35`	`config.logger.warnOnce(`
`36`	`36`	`\n'vite/dynamic-import-polyfill' is no longer needed if you target modern browsers`
`37`	`37`	`)`