-
Notifications
You must be signed in to change notification settings - Fork 12k
Suggestion: running ng init against existing project should only check configuration files #4029
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
Comments
This is actually something we're looking at, yes. This exact same suggestion. I think the main ones are |
@filipesilva , What about the possible breaking changes involved in the angular major version upgrade which may involve changes in the cli blueprints? Why not have a configuration within |
I prefer updating everything with |
@zsfarkas I do the same for now, but find that it can be better to exclude app-related files. |
I've gone through 4 or 5 angular-cli upgrades. These are the only times I recall needing to modifying other files aside from package.json and angular-cli.json NgModule bootstrap change from Angular2 project angular-cli testing changes Two or three versions ago, the unit testing configuration was updated. So +1, it would be nice if the update process didn't go through a diff on all files unless necessary for that release. Also, someone suggested making the command ng update #4007 |
We've removed |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
At the moment, after updating angular-cli packages, running
ng init
would go through all the generated files, including initial NgModules, AppComponent etc, which most likely would be changed by the developer to fit their apps' needs.I think when running
ng init
against existing projects, we should be only asked if we want to update important configuration files that matters to angular-cli.The text was updated successfully, but these errors were encountered: