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 issues: read permissions to docs #1735

Merged

Conversation

minamijoyo
Copy link
Contributor

Fixes #1710

Starting from digger v0.6.41, at least issues: read permission is required to check if PR number is an issue or not. It should be clearly stated in the document so that other folks are not confused.

I'm not sure when issues:write is needed, please add another note where appropriate.

Fixes diggerhq#1710

Starting from digger v0.6.41, at least `issues: read` permission is
required to check if PR number is an issue or not. It should be clearly
stated in the document so that other folks are not confused.

I'm not sure when `issues:write` is needed, please add another note
where appropriate.
@motatoes
Copy link
Contributor

motatoes commented Oct 2, 2024

Hey many thanks for this docs fix 🙏 I would like to add a note that issues: write will be needed for those using drift reconciliation with GitHub but for now this will work for everyone else

@motatoes motatoes self-requested a review October 2, 2024 12:44
@motatoes motatoes merged commit c7f4b3d into diggerhq:develop Oct 2, 2024
1 check was pending
@minamijoyo minamijoyo deleted the add-issues-permissions-to-docs branch October 2, 2024 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Github - 403 for issue permission
2 participants