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

Disallow self-LGTMs #863

Merged
merged 1 commit into from
Jun 1, 2016
Merged

Conversation

hqhq
Copy link
Contributor

@hqhq hqhq commented Jun 1, 2016

As opencontainers/project-template#13
is merged, change pullapprove accordingly.

Signed-off-by: Qiang Huang [email protected]

As opencontainers/project-template#13
is merged, change pullapprove accordingly.

Signed-off-by: Qiang Huang <[email protected]>
@cyphar
Copy link
Member

cyphar commented Jun 1, 2016

Can you also include the MAINTAINERS_GUIDE.md changes from that PR as well?

@hqhq
Copy link
Contributor Author

hqhq commented Jun 1, 2016

@cyphar I think the plan should not be that we sync these docs to every projects of OCI when they changed, if we want to maintain these docs in one place, we might need some action like we did in runtime-spec for code-of-conduct.md, that's something we need to figure out.

@cyphar
Copy link
Member

cyphar commented Jun 1, 2016

If we're going to do that, we should link the files IMO.

LGTM

Approved with PullApprove

@dqminh
Copy link
Contributor

dqminh commented Jun 1, 2016

LGTM

Approved with PullApprove

@dqminh dqminh merged commit 4450f7c into opencontainers:master Jun 1, 2016
@hqhq hqhq deleted the disallow_self_lgtm branch June 2, 2016 01:55
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
…ched

config: Explicily make consoleSize unspecified if terminal is false or unset
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants