webpack it!
Closed this issue · 12 comments
Hey!!! Sean from the webpack team! Would you take a PR if provided to bundle the assets on both main and renderer portions of this application?!
Pros: startup and render performance could be significantly faster!
Cons: have a build step?
I've been looking for a great case study for electron performance and app migration and I think this would be a great one.
Thanks!
Sean - webpack team
I would be willing to setup a CI build in appveyor for your webpack build step.
Sure
In hopes of not biting off more then I can chew, if it would be beneficial to allow a broader collaboration for this, and given a PR would be acceptable, I can create (or update this issue) to include a spec like set of requirements and isolated tasks for contributors to tackle. That way this work isn't just hung up by myself (if it turns out to be a larger task).
Can you please explain to the layman (or if you have any links handy) why using webpack with electron and serving the code through a server instead of consuming it directly would be faster?
I've struggled to understand this ever since I first switched to electron-compile and then directly to babel using babel --watch to auto-transform on changes to the source.
@whphhg I didn't think that webpack required you to serve it through a server? It can be used to just compress the code into single files can't it? Maybe I'm misunderstanding what it does?
@cursorial you are correct, I have misunderstood how it's used all this time. The server is only used for HMR during development and in production it loads the local entry point.
I've found these resources to be useful:
Did somebody pick this up already? I have some spare time tonight / this weekend and I want to look into this.
Hi finally have a bit of time to split some things up here! I'm going to pull the code down first and do a little assessment before getting started.
@TheLarkInn I discovered an issue while trying to implement this feature, check #32 to see the issue I ran into, but it is being worked on.
The re-write is now using webpack/eslint, closing this one.