-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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: vite upgrade #13195
feat: vite upgrade #13195
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Two major issues that we are facing : -
|
We removed the |
Unable to find test scripts. Please add necessary tests to the PR. |
Some Caveats : -
|
Temporarily removing as a reviewer. Add again when PR is ready for review. |
@yaldram has the issue been fixed? taowen/vite-howto#4 |
@taowen Hey, no unfortunately we decided to move forward with cra-5 for now. Inline with this comment - vitejs/vite#7608 (comment) we might have to re-structure our codebase for faster hot reloads. But unfortunately we don't have any docs, guides from the vite team on how to restructure large projects with redux. |
Closing this in favour of #14000 |
Description
Fixes #12066
Type of change
How Has This Been Tested?
Checklist: