-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: support vite 3 #1543
feat: support vite 3 #1543
Conversation
❌ Deploy Preview for vitest-dev failed.
|
@@ -2,7 +2,7 @@ | |||
/// <reference types="vite/client" /> | |||
|
|||
import { defineConfig } from 'vite' | |||
import solid from 'solid-start' // or use 'vite-plugin-solid' instead | |||
import solid from 'vite-plugin-solid' // or use 'vite-plugin-solid' instead |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can remove comment now
@@ -20,6 +20,7 @@ const extraInlineDeps = [ | |||
/@vitest\/dist/, | |||
// Nuxt | |||
'@nuxt/test-utils', | |||
/vite\w*\/dist\/client\/env.mjs/, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is backported to #1546
It seems to take some time to wait for the stable release of Vite 3. Can we just release a major beta version of Vitest? Then we can use Vite 3 in Vitest. Since Vite 2.x doesn't accept feature PR vitejs/vite#8769 now. |
We already support Vite 3, this is PR for updating mostly examples |
Oh, I see. So can we merge this PR #1540 now? |
Sure |
Add support for Vite 3
TODO