Skip to content

Conversation

@dario-piotrowicz
Copy link
Contributor

Description

This PR is trying to address the problem I mentioned here: https://github.com/vitejs/vite/pull/18395/files#r1832458011

(discord thread)

@sapphi-red sapphi-red added p3-downstream-blocker Blocking the downstream ecosystem to work properly (priority) regression The issue only appears after a new release feat: deps optimizer Esbuild Dependencies Optimization feat: environment API Vite Environment API labels Nov 7, 2024
@sapphi-red sapphi-red changed the title feat: allow users to specify their esbuild prebundling platform feat(optimizer): allow users to specify their esbuild platform option Nov 7, 2024
Copy link
Contributor

@hi-ogawa hi-ogawa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I was using esbuildOptions: { banner: undefined } hi-ogawa/vite-environment-examples#148, but it makes sense to be fixed here.

@sapphi-red sapphi-red merged commit 0924879 into vitejs:main Nov 8, 2024
@dario-piotrowicz dario-piotrowicz deleted the dario/prebundling-platform branch November 8, 2024 09:32
moonlitusun pushed a commit to moonlitusun/vite that referenced this pull request May 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat: deps optimizer Esbuild Dependencies Optimization feat: environment API Vite Environment API p3-downstream-blocker Blocking the downstream ecosystem to work properly (priority) regression The issue only appears after a new release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants