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

Add triage and maintain repo permissions #303

Conversation

liamg
Copy link
Contributor

@liamg liamg commented Nov 26, 2019

Closes #279

This PR makes the triage and maintain repository permissions available. See https://github.blog/2019-10-03-delegate-responsibilities-with-expanded-repository-permissions/

@ghost ghost added the size/S label Nov 26, 2019
@kpfleming
Copy link
Contributor

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 :-)

@alexbde
Copy link

alexbde commented Jan 3, 2020

Looks like #296 is another duplicate of this. Please merge one of them :)

@alexbde
Copy link

alexbde commented Jan 3, 2020

Closes #270.

Copy link
Contributor

@majormoses majormoses left a comment

Choose a reason for hiding this comment

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

LGTM

@paultyng paultyng merged commit 192e63b into integrations:master Jan 27, 2020
jcudit pushed a commit that referenced this pull request Feb 24, 2020
Updating docs to include triage and maintain permissions, as per PR #303
@liamg liamg deleted the liamg-add-triage-and-maintain-repo-permissions branch February 26, 2020 14:21
@salvianreynaldi
Copy link

Hi,
the code seems to address both github_team_repository and github_repository_collaborator. However, I get errors when trying to set repository collaborator:

unexpected permission value: triage
unexpected permission value: maintain

Terraform set the permissions correctly, though. However, I can't then modify the permission (e.g. from maintain/triage to pull) in terraform, it gives the same error message, and the permissions are not updated this time.

This doesn't happen with the team repository. Am I missing something?

@paultyng
Copy link
Contributor

@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.

@anGie44
Copy link
Contributor

anGie44 commented May 15, 2020

Fix to be released in the upcoming v2.8.0 release.

kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
…aintain-repo-permissions

Add triage and maintain repo permissions
kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
…ory-docs

Updating docs to include triage and maintain permissions, as per PR integrations#303
kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extra permissions required
8 participants