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

pullapprove: use the right team #848

Merged
merged 1 commit into from
May 27, 2016
Merged

pullapprove: use the right team #848

merged 1 commit into from
May 27, 2016

Conversation

cyphar
Copy link
Member

@cyphar cyphar commented May 27, 2016

On GitHub the maintainers of runC are not the same as the maintainers
of runtime-spec. Fix this, and use the right team.

Fixes #847.

Signed-off-by: Aleksa Sarai [email protected]

@caniszczyk
Copy link
Contributor

Sorry folks, I created a new team call 'runc-mantainers' you can use
https://github.com/orgs/opencontainers/teams/runc-maintainers

@cyphar
Copy link
Member Author

cyphar commented May 27, 2016

Actually, it looks like we need to add a new team just for runC maintainence. In addition, we should probably clean up all of the teams we have in opencontainers -- it doesn't make sense to group them because each project has its own maintainence.

I've updated it to use runc-mainainers -- but that team doesn't current have write access to runC.

On GitHub the maintainers of runC are *not* the same as the maintainers
of runtime-spec. Fix this, and use the right team.

Signed-off-by: Aleksa Sarai <[email protected]>
@hqhq
Copy link
Contributor

hqhq commented May 27, 2016

LGTM

Approved with PullApprove

1 similar comment
@dqminh
Copy link
Contributor

dqminh commented May 27, 2016

LGTM

Approved with PullApprove

@dqminh dqminh merged commit 1ce23a5 into opencontainers:master May 27, 2016
@cyphar cyphar deleted the fix-pullapprove branch May 27, 2016 11:20
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
…ple-header-level

config-windows: Fix header level for network example
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.

5 participants