-
-
Notifications
You must be signed in to change notification settings - Fork 593
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: Add vite #2425
base: alpha
Are you sure you want to change the base?
feat: Add vite #2425
Conversation
I will reformat the title to use the proper commit message syntax. |
Thanks for opening this pull request! |
Nice! Need some help? I'm interested in seeing how this would work with tree shaking. |
@dplewis I think the challenge here is that Vite might/webpack not be compatible with CJS. Any thoughts? |
I’m not familiar with vite but I see you have a umd bundle in your output, you should be able to output cjs. By compatibility are you referring to the input as cjs? We can change that |
Just having some issues getting some of the browser apis (crypto etc) to polyfill to node. seems to be coming along well though |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## alpha #2425 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 64 64
Lines 6256 6232 -24
Branches 1476 1477 +1
=========================================
- Hits 6256 6232 -24 ☔ View full report in Codecov by Sentry. |
Pull Request
Issue
Closes: #2410
Approach
Vite is more modern than webpack, so approaching building files with vite. Testing will still be needed
Tasks