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.

Sign up for GitHub

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

Jump to bottom

fixes #16 - message for 'Could not resolve host' failure #30

Merged
merged 1 commit into from
Sep 4, 2015
Merged

fixes #16 - message for 'Could not resolve host' failure #30

merged 1 commit into from
Sep 4, 2015

Conversation

jantman
Copy link
Owner

@jantman jantman commented Sep 4, 2015

if r10k deploy fails with a 'Could not resolve host' error, append a message explaining how to skip provisioning

@jantman
fixes #16 - if r10k deploy fails with a 'Could not resolve host' erro…
358a9b0 
…r, append a message explaining how to skip provisioning
@jantman
Copy link
Owner Author

jantman commented Sep 4, 2015

PullReview stats

@jantman
Copy link
Owner Author

jantman commented Sep 4, 2015

PullReview Great just fixed 4 issues, 2 to go.
Test Infection Your branch must be test infected. lib/vagrant-r10k/action/deploy.rb:1
Test Infection Your branch must be test infected. spec/acceptance/vagrant-r10k/vagrant-r10k_spec.rb:1
Test Infection Your branch must be test infected. spec/unit/action_deploy_spec.rb:1
Style We ignored double quote rule for this review spec/unit/action_deploy_spec.rb#raises an error

@jantman
Copy link
Owner Author

jantman commented Sep 4, 2015

Acceptance tests passed locally

jantman added a commit that referenced this pull request Sep 4, 2015
@jantman
Merge pull request #30 from /issues/16
d08f8ac 
fixes #16 - message for 'Could not resolve host' failure
@jantman jantman merged commit d08f8ac into master Sep 4, 2015
@jantman jantman deleted the issues/16 branch September 4, 2015 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jantman