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

Added option '--destroy' for the 'reload' command. #5410

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@trensen
Contributor

trensen commented Mar 1, 2015

This PR introduces a new option --destroy to the reload command, which is a shorthand for vagrant destroy --force followed by vagrant up.

Fixes #5378.

Added option '--destroy' for the 'reload' command.
This is a shorhand for 'vagrant destroy --force' followed by 'vagrant up'.
@berendt

This comment has been minimized.

Show comment
Hide comment
@berendt

berendt Mar 24, 2015

Contributor

I would prefer to introduce a new command rebuild like suggested in the linked issue #5378.

Contributor

berendt commented Mar 24, 2015

I would prefer to introduce a new command rebuild like suggested in the linked issue #5378.

@sethvargo

This comment has been minimized.

Show comment
Hide comment
@sethvargo

sethvargo May 30, 2015

Contributor

Hi @trensen

I am going to close this PR as a result of discussion in the original issue. We think this should be a separate command instead of a flag. I would happily review a PR for the additional command 😄. Sorry!

Contributor

sethvargo commented May 30, 2015

Hi @trensen

I am going to close this PR as a result of discussion in the original issue. We think this should be a separate command instead of a flag. I would happily review a PR for the additional command 😄. Sorry!

@sethvargo sethvargo closed this May 30, 2015

@trensen

This comment has been minimized.

Show comment
Hide comment
@trensen

trensen May 31, 2015

Contributor

@sethvargo no problem, I'll propose an additional command then!
Just to be sure: should it be rebuild?

Contributor

trensen commented May 31, 2015

@sethvargo no problem, I'll propose an additional command then!
Just to be sure: should it be rebuild?

@sethvargo

This comment has been minimized.

Show comment
Hide comment
@sethvargo

sethvargo May 31, 2015

Contributor

@trensen I think someone beat you to it: #5613 😄

Contributor

sethvargo commented May 31, 2015

@trensen I think someone beat you to it: #5613 😄

@trensen

This comment has been minimized.

Show comment
Hide comment
@trensen

trensen May 31, 2015

Contributor

@sethvargo Ah, too bad for me, but great to have such command being introduced! 😊

Contributor

trensen commented May 31, 2015

@sethvargo Ah, too bad for me, but great to have such command being introduced! 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment