Skip to content

Minor doc edit to add clarity around the --volume path format#13776

Merged
moxiegirl merged 1 commit intomoby:masterfrom
duglin:MinorVolDocEdit
Jun 6, 2015
Merged

Minor doc edit to add clarity around the --volume path format#13776
moxiegirl merged 1 commit intomoby:masterfrom
duglin:MinorVolDocEdit

Conversation

@duglin
Copy link
Copy Markdown
Contributor

@duglin duglin commented Jun 5, 2015

Also add a comment to the ValidatePath func so devs/reviewers
know exactly what its looking for.

Signed-off-by: Doug Davis [email protected]

@duglin
Copy link
Copy Markdown
Contributor Author

duglin commented Jun 5, 2015

ping @moxiegirl @fredlf @SvenDowideit

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tics not quotes please. Sorry, I see this was let through before and you were just following a bad precedent.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yep - just following someone else's lead :-) fixed.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Given that this approach (ie, a code-block describing the flag/option) is already used extensively in this document, I think this is fine (for consistency).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@duglin LOL

Also add a comment to the ValidatePath func so devs/reviewers
know exactly what its looking for.

Signed-off-by: Doug Davis <[email protected]>
@duglin duglin force-pushed the MinorVolDocEdit branch from c2b9270 to 3fcf53d Compare June 5, 2015 22:11
@thaJeztah
Copy link
Copy Markdown
Member

LGTM, thanks @duglin

@moxiegirl
Copy link
Copy Markdown
Contributor

LGTM

moxiegirl pushed a commit that referenced this pull request Jun 6, 2015
Minor doc edit to add clarity around the --volume path format
@moxiegirl moxiegirl merged commit ffd74e7 into moby:master Jun 6, 2015
@moxiegirl
Copy link
Copy Markdown
Contributor

danke @duglin

@thaJeztah
Copy link
Copy Markdown
Member

Oh, i18n thanking! :)

Dankjewel, @duglin

@moxiegirl
Copy link
Copy Markdown
Contributor

@jfrazelle cherry pick on this one please

@jessfraz
Copy link
Copy Markdown
Contributor

jessfraz commented Jun 9, 2015

cherry-pciked

@duglin duglin deleted the MinorVolDocEdit branch July 10, 2015 14:52
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.

5 participants