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

Improve formatting of overwrite error #227

Merged

Conversation

nick-stroud
Copy link
Collaborator

@nick-stroud nick-stroud commented Apr 21, 2022

Previous cli output:

$ ./ghpc create  examples/my_blueprint.yaml 
2022/04/21 14:08:15 Failed to overwrite existing blueprint. Use the -w command line argument to enable overwrite. If overwrite is already enabled then this may be because you are attempting to remove a resource group, which is not supported : The directory already exists: my_blueprint

Updated output

$ ./ghpc create  examples/my_blueprint.yaml 

Failed to overwrite existing blueprint.

Use the -w command line argument to enable overwrite.
If overwrite is already enabled then this may be because you are attempting to remove a resource group, which is not supported.
original error: The directory already exists: my_blueprint

Submission Checklist

  • Have you installed and run this change against pre-commit? pre-commit install
  • Are all tests passing? make tests
  • If applicable, have you written additional unit tests to cover this
    change?
  • Is unit test coverage still above 80%?
  • Have you updated any application documentation such as READMEs and user
    guides?
  • Have you followed the guidelines in our Contributing document?

@heyealex heyealex assigned nick-stroud and unassigned heyealex Apr 21, 2022
@nick-stroud nick-stroud merged commit c42ab26 into GoogleCloudPlatform:develop Apr 21, 2022
@nick-stroud nick-stroud deleted the improve_overwrite_error branch April 28, 2022 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants