Skip to content

fix: Dont require Vite#877

Open
timfish wants to merge 3 commits intogetsentry:mainfrom
timfish:timfish/fix/dont-require-vite
Open

fix: Dont require Vite#877
timfish wants to merge 3 commits intogetsentry:mainfrom
timfish:timfish/fix/dont-require-vite

Conversation

@timfish
Copy link
Collaborator

@timfish timfish commented Feb 16, 2026

Not sure I really like this solution as it is. We should probably at least add Vite as a peer dependency otherwise it may fail to load it when used when dependencies are strictly enforces, like with pnpm 🤔

@github-actions
Copy link

github-actions bot commented Feb 16, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • Rollup/Vite no longer uses unplugin by timfish in #858
  • Esbuild no longer uses unplugin by timfish in #871
  • Webpack no longer uses unplugin by timfish in #870

Bug Fixes 🐛

  • (webpack) Deduplicate webpack deploys by chargome in #875
  • Dont require Vite by timfish in #877

🤖 This preview updates automatically when you update the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vite: The CJS build of Vite's Node API is deprecated

1 participant