-
Notifications
You must be signed in to change notification settings - Fork 6
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
⬆️ Upgrade to Vite 6 #758
base: main
Are you sure you want to change the base?
⬆️ Upgrade to Vite 6 #758
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #758 +/- ##
==========================================
- Coverage 80.05% 80.02% -0.03%
==========================================
Files 238 238
Lines 4953 4951 -2
Branches 1350 1341 -9
==========================================
- Hits 3965 3962 -3
- Misses 958 959 +1
Partials 30 30 ☔ View full report in Codecov by Sentry. |
Blocked by storybookjs/storybook#29719 :) |
cd9f403
to
9607a25
Compare
Released a couple of weeks ago :) This currently requires us to force storybook vite builder to resolve esbuild to 0.24.0, despite vite 6.0.5 pinning this dependency. See storybookjs/storybook#30115
9607a25
to
d5c38bb
Compare
Workaround applied from pinned Storybook issue. |
@robinmolen @vaszig considering this is build tooling and CI is passing, I think it's not too extreme to merge this in main before the 2.5.0 release, but feel free to disagree if you think it's too risky! |
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.
I am not familiar with this, especially the new version, but if Robin agrees of course we can merge it!
Released a couple of weeks ago :)