Skip to content
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

Ionic should not install all dependencies unless instructed to #5032

Open
Baraka-Mugisha opened this issue Aug 9, 2023 · 0 comments
Open
Labels

Comments

@Baraka-Mugisha
Copy link

I would like to point out that it is unfortunate that the ionic start command automatically executes the npm install command to install all npm dependencies. Given also that the options for skipping dependencies does not work

In my opinion, the command's main purpose should be to generate code, not handle dependency installation. It would be more in line with the single responsibility principle if the command focused solely on code generation.

Running npm install is a simple command that we can manually execute after generating the project. I mention this because the automatic execution of npm install during ionic start can significantly increase the execution time, especially on slow networks. This can be frustrating, especially when we need to run the command multiple times to explore different templates or options.

It's important to note that generating code does not necessarily mean that we will immediately execute it. Separating the dependency installation step from the code generation step would provide more flexibility and control over the development process

@ionitron-bot ionitron-bot bot added the triage label Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant