-
Notifications
You must be signed in to change notification settings - Fork 166
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
Make Webpack faster #210
Comments
What
Looks like much time are wasted on I/O, copying files every time. |
Indeed, it looks likes it's copying a bunch of useless stuff too. https://github.com/PrismarineJS/prismarine-web-client/blob/master/webpack.common.js#L72 also copy sub-directories while we are only interested in the .png at the root (and maybe some entities and gui textures, but we clearly don't need the many many individual files of the blocks). |
Does it copy the files for dev builds though? I thought those were incremental |
So, I wrote a little plugin for symlinking directories, now dev builds are faster
|
#227 was reverted so we're back at square 1 |
yep, I reverted because it broke npm start |
Hi all! I investigated new ways to improve build times. Here's what I found:
|
switching to esbuild seems good I think it'd be pretty helpful to first switch to that here |
Now, one year later I have investigated new ways to improve build times once again and have completely moved to Rsbuild! After a month of usage, I got feedback from different devs and everyone is really happy as it is performing well on any platform and by almost 2x faster than Vite. Unlike esbuild it supports HMR, code splitting, Webpack plugins, and fast incremental builds. So in my case compared to esbuild it was:
I bet Rsbuild will stay the fastest bundler for a few more years. Can't think anymore of something better... |
We're at this point again. Somehow, webpack is again being extremely slow. This is hampering development; I can't iterate quickly/test stuff if I need to wait a minute every time I make a change, no matter how small.
The text was updated successfully, but these errors were encountered: