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. |
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
triageandmaintainrepository permissions available. See https://github.blog/2019-10-03-delegate-responsibilities-with-expanded-repository-permissions/