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

refactor(@angular/cli): remove ng init & ng update #4628

Merged
merged 1 commit into from
Feb 12, 2017

Conversation

Brocco
Copy link
Contributor

@Brocco Brocco commented Feb 11, 2017

BREAKING CHANGE: Removing the ng init & ng update commands because their current implementation causes more problems than it solves. Update functionality will return to the CLI, until then manual updates of applications will need done.

@hansl
Copy link
Contributor

hansl commented Feb 11, 2017

At least a BREAKING CHANGE section on this one please :)

@Brocco
Copy link
Contributor Author

Brocco commented Feb 11, 2017

Thank you, breaking change text has been added to the PR.

Copy link
Contributor

@hansl hansl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll reword the BREAKING CHANGE part. Thanks! LGTM.

@alaindeurveilher
Copy link

Hi,
just for information, as of today, the main README.md has not been updated accordingly to the realase of the 1.0.0-beta.32, and still suggest to run ng update after an update of @angular/cli.

@filipesilva
Copy link
Contributor

@AlainD- you're absolutely right, thanks for the heads up!

@filipesilva
Copy link
Contributor

filipesilva commented Feb 17, 2017

Here's a PR that'll fix it #4771

filipesilva added a commit that referenced this pull request Feb 17, 2017
`ng update` has been removed in #4628.

Close #4771
@fvilers
Copy link

fvilers commented Feb 19, 2017

I'm missing the init command. How can I create an application in an existing git repository?

@equinox
Copy link
Contributor

equinox commented Feb 19, 2017

@fvilers You could ng new appname --skip-git (optionally --skip-install also to skip yarn/npm install to save moving the node_modules over also) a new application in another directory then move the files over manually.

@fvilers
Copy link

fvilers commented Feb 19, 2017

@equinox This is what I did, but this is not the behavior I expect from a cli. I hope the maintainers will restore the command soon.

@kuncevic
Copy link

kuncevic commented Feb 19, 2017

@fvilers you are not alone #4768
It might come back or might not, basically the core team do not want to support such dunctionality but if anyone from community can come upnwith a fix they are looking forward to it.

@Brocco Brocco deleted the remove-ng-init-update branch February 20, 2017 03:57
asnowwolf pushed a commit to asnowwolf/angular-cli that referenced this pull request Apr 12, 2017
BREAKING CHANGE: Removing the `ng init` & `ng update` commands because their current implementation causes more problems than it solves. Once RC is released, we won't need to use those to update anymore as the step will be as simple as installing the latest version of the CLI.
asnowwolf pushed a commit to asnowwolf/angular-cli that referenced this pull request Apr 12, 2017
@ctilley83
Copy link

So what's the solution for upgrading an existing app guys? Manually change the package.json file?

@alaindeurveilher
Copy link

I don't know the recommendations but what I usually do, I create a new project in a temporary folder, and then compare the differences between that new project and mine. I manually modify my project accordingly then.

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants