Akryum / meteor-vite

MIT License
34 stars 19 forks source link

'use esbuild instead of babel' #35

Closed WayneUong closed 3 months ago

StorytellerCZ commented 3 months ago

Why is this needed?

JorgenVatle commented 3 months ago

Thank you for the contribution. We've made some changes to meteor-vite that supersedes some of the changes suggested here.

If I recall correctly, this stage of the build process didn't really amount to any noticeable time to complete, even for larger apps.

With the upcoming 3.0.0 release, this step has become redundant and is omitted entirely now that builds and client-side assets are handled entirely by Vite. This has sped up build times quite significantly now that Meteor now only has to build a minimal client bundle with core packages, which seems to have been the biggest bottleneck in build times. I'll update the PR with some benchmarks as soon as I have the chance. 👍