Skip to content

Add triage and maintain repo permissions#303

Merged
paultyng merged 2 commits intointegrations:masterfrom
liamg:liamg-add-triage-and-maintain-repo-permissions
Jan 27, 2020
Merged

Add triage and maintain repo permissions#303
paultyng merged 2 commits intointegrations:masterfrom
liamg:liamg-add-triage-and-maintain-repo-permissions

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
Collaborator

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