denali-js / cli

The official CLI for starting, building, and running Denali apps and addons
MIT License
5 stars 5 forks source link

Significantly improve CLI startup times #42

Closed davewasmer closed 6 years ago

davewasmer commented 6 years ago

Before

image

After

image


Move a few heavier modules from static imports to lazy require()s. Also remove support for globally installed addons via yarn and npm. If Yarn is detected, Denali will search Yarn for global addons, otherwise it wills search npm, but not both. This lets Yarn users avoid the delays that npm forces (because we need to shell out for npm root, which eats 500ms).

We could make this behavior configurable in the future, once we have a ~/.denali config file