search

binbashar / leverage

Binbash Leverage CLI intended to orchestrate Leverage Reference Architecture for AWS (www.binbash.co/leverage)
https://pypi.org/project/leverage/
Apache License 2.0
17 stars 2 forks source link

Enhancement | Improve error message when no TERRAFORM_IMAGE_TAG #167

Closed juanmatias closed 1 year ago

juanmatias commented 1 year ago

Describe the Enhance

When no TERRAFORM_IMAGE_TAG is set this is the error:

╰─ leverage aws configure sso
[14:49:39.071] ERROR    No docker image tag defined.
                        Please set `TERRAFORM_IMAGE_TAG` variable before running a Leverage command.

This message could be improved, e.g. saying this variable has to be set in build.env file.

Expected Behavior

Messages should be self explanatories.

Use Case

A user with the var not set.

Describe Ideal Solution

n/a

Alternatives Considered

n/a

Additional Context

https://github.com/binbashar/leverage/blob/9f76b0ea278931913be3967155bf9f3b2de3496b/leverage/container.py#L92-L95