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

remove update command in nestjs v9 #1696

Closed
1 task done
micalevisk opened this issue Jun 24, 2022 · 1 comment · Fixed by #1699
Closed
1 task done

remove update command in nestjs v9 #1696

micalevisk opened this issue Jun 24, 2022 · 1 comment · Fixed by #1699
Labels

Comments

@micalevisk
Copy link
Member

micalevisk commented Jun 24, 2022

Is there an existing issue that is already proposing this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe it

taking this #1204 (comment)

We might need to remove the update command entirely soon as it isn't even close to a real "updating/upgrading" feature. I'd recommend using npx ncu -u instead (for updating deps).

I guess we can do that in v9 😄

Describe the solution you'd like

either

  • drop the command update from @nestjs/cli and update the migration guide to clarify why,
  • or display a deprecating warning message when nest update is being used, to finally remove it in v10

Teachability, documentation, adoption, migration strategy

it depends on which path above we will follow

What is the motivation / use case for changing the behavior?

people often complains about nest update not working as they expected it should, and I don't think the CLI need that feature because there are other tools for such task

@kamilmysliwiec
Copy link
Member

Would you like to create a PR for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants