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

Apply command prints names of parameters with missing values #322

Merged
merged 1 commit into from Mar 20, 2020

Conversation

@orien
Copy link
Member

orien commented Mar 19, 2020

It'd be handy two know which parameter is missing a value when stack_master apply fails. Let's print this out in the error message.

Empty/blank parameters detected. Please provide values for these parameters:
 - param1
 - param2
Parameters will be read from files matching the following globs:
 - parameters/myapp.y*ml
 - parameters/us-east-1/myapp.y*ml
Copy link
Contributor

petervandoros left a comment

❤️

@orien orien merged commit 78cfbd7 into master Mar 20, 2020
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@orien orien deleted the list-blank-params branch Mar 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.