Optimize 'electronize start' faster #13

Closed
opened 2026-01-29 16:27:31 +00:00 by claunia · 2 comments
Owner

Originally created by @GregorBiswanger on GitHub (Oct 19, 2017).

Originally assigned to: @robertmuehsig on GitHub.

Install npm packages only for one time.. possibly on 'electronize init'?!

Originally created by @GregorBiswanger on GitHub (Oct 19, 2017). Originally assigned to: @robertmuehsig on GitHub. Install npm packages only for one time.. possibly on 'electronize init'?!
claunia added the Feature label 2026-01-29 16:27:31 +00:00
Author
Owner

@robertmuehsig commented on GitHub (Oct 19, 2017):

"init" ist currently only used for the manifest file and "init" currently doesn't know anything about the target platform that we want to run and it doesn't know the obj/desktop folder construct, so... this would be not a good fit IMHO.

But: We can make the build and start command much smarter. In theory (with a bit of risk of course), if we detect the npm_modules folder, we may just skip npm install.

At first I need to do this one: https://github.com/ElectronNET/Electron.NET/issues/7

The rest of the idea is discussed there.

@robertmuehsig commented on GitHub (Oct 19, 2017): "init" ist currently only used for the manifest file and "init" currently doesn't know anything about the target platform that we want to run and it doesn't know the obj/desktop folder construct, so... this would be not a good fit IMHO. But: We can make the build and start command much smarter. In theory (with a bit of risk of course), if we detect the npm_modules folder, we may just skip npm install. At first I need to do this one: https://github.com/ElectronNET/Electron.NET/issues/7 The rest of the idea is discussed there.
Author
Owner

@robertmuehsig commented on GitHub (Nov 7, 2017):

Maybe not ideal, but if we detect the node_modules folder, we skip npm install... which speeds things up.

@robertmuehsig commented on GitHub (Nov 7, 2017): Maybe not ideal, but if we detect the node_modules folder, we skip npm install... which speeds things up.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/Electron.NET#13