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

config: Simplify title to "Configuration" #901

Merged
merged 1 commit into from
Aug 18, 2017

Conversation

wking
Copy link
Contributor

@wking wking commented Jul 19, 2017

The capitalization in “Container Configuration file” (which we've used since #176) was halfway between “Title Case” and “Sentence case”. The current spec isn't particularly consistent (e.g. we have both “Specification version” and “POSIX-platform Mounts”), but the ToC has used “Configuration” for this file since #626, so dodge the sentence/title issue and use that here too.

The capitalization in "Container Configuration file" (which we've used
since 70372d3, *.md: update TOC and links, 2015-09-10, opencontainers#176) was
halfway between "Title Case" and "Sentence case".  The current spec
isn't particularly consistent (e.g. we have both "Specification
version" and "POSIX-platform Mounts"), but the ToC has used
"Configuration" for this file since e7be40f (Cleanup the spec a bit
to remove WG/git text that's not really part of the spec, 2016-11-14,
opencontainers#626) so dodge the sentence/title issue and use that here too.

Signed-off-by: W. Trevor King <wking@tremily.us>
@hqhq
Copy link
Contributor

hqhq commented Aug 18, 2017

LGTM

Approved with PullApprove

1 similar comment
@crosbymichael
Copy link
Member

crosbymichael commented Aug 18, 2017

LGTM

Approved with PullApprove

@crosbymichael crosbymichael merged commit b747937 into opencontainers:master Aug 18, 2017
@wking wking deleted the config-title branch August 22, 2017 19:09
@vbatts vbatts mentioned this pull request Sep 8, 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.

3 participants