Skip to content
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

Closed
wants to merge 3 commits into from
Closed

feat: support vite 3 #1543

wants to merge 3 commits into from

Conversation

sheremet-va
Copy link
Member

Add support for Vite 3

TODO

  • Updated dependencies to stable versions, when Vite 3 releases

@sheremet-va sheremet-va self-assigned this Jun 25, 2022
@netlify
Copy link

netlify bot commented Jun 25, 2022

Deploy Preview for vitest-dev failed.

Name Link
🔨 Latest commit fadab92
🔍 Latest deploy log https://app.netlify.com/sites/vitest-dev/deploys/62b6daf1779ae8000adc9bbd

@sheremet-va sheremet-va marked this pull request as draft June 25, 2022 09:36
@@ -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
Copy link
Member Author

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/,
Copy link
Member Author

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

@sxzz
Copy link
Contributor

sxzz commented Jul 1, 2022

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.

@sheremet-va
Copy link
Member Author

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

@sxzz
Copy link
Contributor

sxzz commented Jul 1, 2022

Oh, I see. So can we merge this PR #1540 now?

@sheremet-va
Copy link
Member Author

Oh, I see. So can we merge this PR #1540 now?

Sure

@sheremet-va sheremet-va deleted the feat/vite-3 branch December 7, 2022 08:44
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.

None yet

2 participants