-
Notifications
You must be signed in to change notification settings - Fork 763
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
Add triage and maintain repo permissions #303
Add triage and maintain repo permissions #303
Conversation
Note that this PR contains the same (unrelated) fix as in #310. We really need that one merged so everyone can stop making this same change in their branches :-) |
Looks like #296 is another duplicate of this. Please merge one of them :) |
Closes #270. |
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.
LGTM
Updating docs to include triage and maintain permissions, as per PR #303
Hi,
Terraform set the permissions correctly, though. However, I can't then modify the permission (e.g. from maintain/triage to This doesn't happen with the team repository. Am I missing something? |
@salvianreynaldi not that I'm aware of, this is working for me now, but I do occasionally get unexpected diffs with different permission strings, but it doesn't seem to affect the actual runtime permissions available to people. I haven't dug in to the diff/read issue yet to figure out the reason for that, it may be related. If you can though, I'd recommend opening a new issue with a repro if you have one, or at least a log. |
I believe something needs to be changed either here: Will try to play around. API reference: https://github.com/terraform-providers/terraform-provider-github/blob/8c9a8c3892278e7a32fda6b8744190994722db33/github/util_permissions.go#L51 Also this needs adjusting: https://github.com/romaninsh/terraform-provider-github/blob/master/website/docs/r/repository_collaborator.html.markdown#argument-reference |
Fix to be released in the upcoming |
…aintain-repo-permissions Add triage and maintain repo permissions
…ory-docs Updating docs to include triage and maintain permissions, as per PR integrations#303
Closes #279
This PR makes the
triage
andmaintain
repository permissions available. See https://github.blog/2019-10-03-delegate-responsibilities-with-expanded-repository-permissions/