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

Add error messages for unset variables #15

Merged
merged 1 commit into from May 20, 2019

Conversation

@dmrty
Copy link
Contributor

commented May 17, 2019

This is a first try at correcting the request #14.

  • Change all the .yml files within Docs with using ${VARIABLE_NAME?Variable VARIABLE_NAME not set}
  • Did not change them when default value was provided.

@dmrty dmrty changed the title Add error messages for unset variables (Answers #14) Add error messages for unset variables May 17, 2019

@tiangolo tiangolo merged commit 76fd556 into tiangolo:master May 20, 2019

2 of 5 checks passed

Header rules - dockerswarm-rocks No header rules processed
Details
Pages changed - dockerswarm-rocks 13 new files uploaded
Details
Redirect rules - dockerswarm-rocks No redirect rules processed
Details
Mixed content - dockerswarm-rocks No mixed content detected
Details
netlify/dockerswarm-rocks/deploy-preview Deploy preview ready!
Details
@tiangolo

This comment has been minimized.

Copy link
Owner

commented May 20, 2019

Great! Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.