Skip to content
This repository has been archived by the owner on Apr 6, 2023. It is now read-only.

feat(vite): upgrade to vite 2.9.0 #3974

Merged
merged 6 commits into from Mar 31, 2022
Merged

feat(vite): upgrade to vite 2.9.0 #3974

merged 6 commits into from Mar 31, 2022

Conversation

danielroe
Copy link
Member

@danielroe danielroe commented Mar 30, 2022

❓ Type of change

  • 🧹 Chore

📚 Description

This PR upgrades to vite 2.9.0, which has a raft of new features and improvements.

Notably it adds non-blocking pre bundling of dependencies, which I think means we can drop our own pre-warming fo the server.

Note: there is currently an issue with our vite-node integration. I suspect @antfu is already on it.

@netlify
Copy link

netlify bot commented Mar 30, 2022

Deploy Preview for nuxt3-docs canceled.

Name Link
🔨 Latest commit 5f3f7fc
🔍 Latest deploy log https://app.netlify.com/sites/nuxt3-docs/deploys/6245730c9e0cc60008d29cb6

@pi0
Copy link
Member

pi0 commented Mar 30, 2022

Notably it adds vitejs/vite#6758, which I think means we can drop our own pre-warming fo the server.

Are you sure it also triggers build warmup for entry or just doing same prebundling (of client) in background?

pi0 added a commit that referenced this pull request Mar 30, 2022
@antfu antfu self-requested a review March 31, 2022 06:10
@antfu
Copy link
Member

antfu commented Mar 31, 2022

The error is not related to Vite 2.9, but rather a misalignment that only happens on stub mode (build mode works fine). I'll investigate the issue in another PR (update: #3983) but I think it shouldn't block the improvements of having Vite 2.9.

warm up

The optimization in 2.9 becomes lazier and non-blocking, while I guess our warm-up could still benefit the initial loading time. I think we could keep it.

@antfu antfu marked this pull request as ready for review March 31, 2022 06:54
antfu
antfu previously approved these changes Mar 31, 2022
@antfu antfu self-requested a review March 31, 2022 06:54
@antfu antfu dismissed their stale review March 31, 2022 06:55

dismiss

@danielroe
Copy link
Member Author

Thanks @antfu! I'll restore the warm up.

@pi0 pi0 changed the title chore(vite): upgrade to vite 2.9.0 feat(vite): upgrade to vite 2.9.0 Mar 31, 2022
@pi0 pi0 merged commit 5b2ed85 into main Mar 31, 2022
@pi0 pi0 deleted the chore/vite-2.9 branch March 31, 2022 09:28
@danielroe danielroe added the 3.x label Jan 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants