Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update CI ACL #15

Merged
merged 1 commit into from
Jun 21, 2017
Merged

Update CI ACL #15

merged 1 commit into from
Jun 21, 2017

Conversation

talex5
Copy link
Contributor

@talex5 talex5 commented Jun 19, 2017

Note: it would be better to check whether the user has write access on the repository, but that's currently blocked on mirage/ocaml-github#185.

Signed-off-by: Thomas Leonard thomas.leonard@docker.com

src/ci.ml Outdated
@@ -106,7 +106,8 @@ let can_build =
| `Production ->
any [
username "admin";
username "github:rneugeba";
username "github:rn";
username "github:samoht";
Copy link
Member

Choose a reason for hiding this comment

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

rn is in the linuxkit org

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point. I'll remove that entry, then.

Note: it would be better to check whether the user has write access on
the repository, but that's currently blocked on
mirage/ocaml-github#185.

Signed-off-by: Thomas Leonard <thomas.leonard@docker.com>
@talex5 talex5 merged commit c493d22 into linuxkit:master Jun 21, 2017
@talex5 talex5 deleted the acl branch June 21, 2017 14:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants