Skip to content

Conversation

@Mashimiao
Copy link

double quotes are not necessary when setting env value

Signed-off-by: Ma Shimiao mashimiao.fnst@cn.fujitsu.com

@Mashimiao Mashimiao changed the title config.md: use VARNAME instead config.md: small fixes May 10, 2017
double quotes are not necessary when setting env value
and other sentences fix

Signed-off-by: Ma Shimiao <mashimiao.fnst@cn.fujitsu.com>
@philips
Copy link
Contributor

philips commented May 11, 2017

LGTM

Approved with PullApprove


- **Env** *array of strings*, OPTIONAL

Entries are in the format of `VARNAME="var value"`.
Copy link
Contributor

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.
Copy link
Contributor

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.
Copy link
Contributor

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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"may be replaced"

@stevvooe
Copy link
Contributor

stevvooe commented May 11, 2017

LGTM

@Mashimiao I'm going to merge this, but please fix the items in a follow up.

Approved with PullApprove

@stevvooe stevvooe merged commit 13eeafe into opencontainers:master May 11, 2017
@Mashimiao
Copy link
Author

OK, fixed in #673

@vbatts vbatts mentioned this pull request May 19, 2017
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.

4 participants