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

schema: Remove string pointers #656

Merged
merged 3 commits into from
Jan 18, 2017

Conversation

wking
Copy link
Contributor

@wking wking commented Jan 13, 2017

And update the style guide to catch up with #653.

They've changed their anchor pattern for inline comments.

Signed-off-by: W. Trevor King <wking@tremily.us>
The maintainer position on this has shifted, with the sea change
coming around [1].

[1]: http://ircbot.wl.linuxfoundation.org/meetings/opencontainers/2017/opencontainers.2017-01-11-22.04.log.html#l-108

Signed-off-by: W. Trevor King <wking@tremily.us>
Catch up with 868e631 (Remove string pointers, 2017-01-12, opencontainers#653).

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

hqhq commented Jan 14, 2017

LGTM

Approved with PullApprove

1 similar comment
@mrunalp
Copy link
Contributor

mrunalp commented Jan 18, 2017

LGTM

Approved with PullApprove

@mrunalp mrunalp merged commit b12752e into opencontainers:master Jan 18, 2017
@wking wking deleted the string-pointer-fixups branch January 19, 2017 05:05
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