Skip to content

README: Use relative links to the proposals #51

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

Merged
merged 1 commit into from
Apr 6, 2018

Conversation

wking
Copy link
Contributor

@wking wking commented Apr 4, 2018

There's no need to use absolute links for these. And using relative links sets a good example for future proposals, because the links will work in the proposal branch's README before the referenced proposal lands in the TOB master.

There's a similar absolute link for the code-of-conduct in the distribution proposal. The intention may have been to show the absolute URL which would be included in new project's repository. I don't think that matters, because the project template has included a similar link since opencontainers/project-template@ee72bc89 (opencontainers/project-template#45), so I've left that alone here.

There's no need to use absolute links for these.  And using relative
links sets a good example for future proposals, because the links will
work in the proposal branch's README before the referenced proposal
lands in the TOB master.

There's a similar absolute link for the code-of-conduct in the
distribution proposal.  The intention may have been to show the
absolute URL which would be included in new project's repository.  I
don't think that matters, because the project template has included a
similar link since opencontainers/project-template@ee72bc89
(CONTRIBUTING: Code of conduct, meetings, mailing list, and IRC,
2016-09-08, opencontainers#45), so I've left that alone here.

Signed-off-by: W. Trevor King <[email protected]>
@wking wking force-pushed the readme-relative-links branch from f8ff9a4 to d9708b5 Compare April 4, 2018 23:16
Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

looks good to me..

@caniszczyk caniszczyk merged commit fe745a3 into opencontainers:master Apr 6, 2018
@wking wking mentioned this pull request May 10, 2018
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