-
Notifications
You must be signed in to change notification settings - Fork 11
fix some permissions in a few rust repos #85
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
Conversation
The permissions here look like they're generally a mess, but this should make the situation a little better.
|
The following access changes will be introduced as a result of applying the plan: Access Changes |
|
Before merge, verify that all the following plans are correct. They will be applied as-is after the merge. Terraform plansipld |
BigLep
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cc @rvagg , but I don't see any concerns here. I'll merge.
|
Actually, since this is IPLD, I don't have permissions to merge (see https://github.com/ipld/github-mgmt/blob/master/github/ipld.yml#L1515 ). I'll leave to @rvagg |
Summary
The permissions here look like they're generally a mess, but this should make the situation a little better. This generally:
Why do you need this?
I need to be able to push to some rust repos and am on the rust team.
What else do we need to know?
DRI: @BigLep
(because I can't merge this)
Reviewer's Checklist