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-linux: Use the implicit link name shortcut #827

Merged

Conversation

wking
Copy link
Contributor

@wking wking commented May 15, 2017

#826 added a redundant link name for libseccomp. This commit restores the old implicit link name shortcut there and updates the other entries (all of which were for the filesystems table in config-linux) that could also use that shortcut. This is obviously not a major change, but I think it makes the source easier to read by minimizing the markup so you can focus on the text.

727e64d (config-linux.md: fix wrong link description about cgroups,
2017-05-14, opencontainers#826) added a redundant link name for libseccomp.  This
commit restores the old implicit link name shortcut [1] there and
updates the other entries (all of which were for the filesystems table
in config-linux) that could also use that shortcut.  This is obviously
not a major change, but I think it makes the source easier to read by
minimizing the markup so you can focus on the text.

[1]: https://daringfireball.net/projects/markdown/syntax#link

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

LGTM

@hqhq
Copy link
Contributor

hqhq commented May 17, 2017

LGTM

Approved with PullApprove

1 similar comment
@crosbymichael
Copy link
Member

crosbymichael commented Jun 1, 2017

LGTM

Approved with PullApprove

@crosbymichael crosbymichael merged commit 0239d87 into opencontainers:master Jun 1, 2017
@wking wking deleted the implicit-link-name-shortcut branch June 2, 2017 04:18
@vbatts vbatts mentioned this pull request Jul 5, 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.

5 participants