This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Raise a VagrantError rather than a RuntimeError for clearer message #1237

Merged

geerlingguy merged 1 commit into geerlingguy:master from oxyc:error-type

Mar 19, 2017

Collaborator

oxyc commented Mar 19, 2017 •

edited

Loading

I thought this was nicer and less confusing, what do you think?

old

$ vagrant status
There was an error loading a Vagrantfile. The file being loaded
and the error message are shown below. This is usually caused by
a syntax error.

Path: /Users/cindy/Projects/Personal/drupal-vm/Vagrantfile
Line number: 77
Message: RuntimeError: You must update Ansible to at least 2.2 to use this version of Drupal VM.

new

$ vagrant status
You must update Ansible to at least 2.2 to use this version of Drupal VM.


          Raise a VagrantError rather than a RuntimeError for clearer message

a6131ce

Owner

geerlingguy commented Mar 19, 2017

💯

geerlingguy merged commit 0a530cb into geerlingguy:master

oxyc deleted the error-type branch

April 24, 2017 15:59

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

Labels

None yet