-
Notifications
You must be signed in to change notification settings - Fork 800
config.md: small fixes #669
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
Conversation
00fb55d to
fac8c72
Compare
double quotes are not necessary when setting env value and other sentences fix Signed-off-by: Ma Shimiao <mashimiao.fnst@cn.fujitsu.com>
fac8c72 to
f8af5bc
Compare
|
|
||
| - **Env** *array of strings*, OPTIONAL | ||
|
|
||
| Entries are in the format of `VARNAME="var value"`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is actually a good catch. These are NOT quoted.
Good job!
|
|
||
| A list of arguments to use as the command to execute when the container starts. | ||
| This value acts as a default and is replaced by an entrypoint specified when creating a container. | ||
| These values act as default and are replaced by an entrypoint specified when creating a container. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"may be replaced".
|
|
||
| A list of arguments to use as the command to execute when the container starts. | ||
| This value acts as a default and is replaced by an entrypoint specified when creating a container. | ||
| These values act as default and are replaced by an entrypoint specified when creating a container. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"as defaults"
|
|
||
| Default arguments to the entrypoint of the container. | ||
| These values act as defaults and are replaced with any specified when creating a container. | ||
| These values act as defaults and are replaced by any specified when creating a container. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"may be replaced"
|
LGTM @Mashimiao I'm going to merge this, but please fix the items in a follow up. |
|
OK, fixed in #673 |
double quotes are not necessary when setting env value
Signed-off-by: Ma Shimiao mashimiao.fnst@cn.fujitsu.com