Skip to content

Request PR write permission for guidance comments #23

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.

Sign up for GitHub

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

Merged
maryliag merged 1 commit into from
Jun 26, 2026
+7 −7
Merged

Request PR write permission for guidance comments #23

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions .github/scripts/pull-request-dashboard/WEBHOOK_SETUP.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ Repository permissions:
- Contents: read-only
- Issues: read and write
- Metadata: read-only
- Pull requests: read-only
- Pull requests: read and write

Organization permissions:

Expand All @@ -63,14 +63,14 @@ Permission rationale:
| ---------- | ------ | ---------------- |
| Checks | Read | Required to subscribe to check-suite events and to read check data for dashboard rows. |
| Contents | Read | Reads PR commits and repository metadata needed by pull/commit APIs. |
| Issues | Read and write | Finds/creates/updates the dashboard issue and posts review-guidance comments on PRs. |
| Issues | Read and write | Finds/creates/updates the dashboard issue and reads existing PR conversation comments before posting review guidance. |
| Metadata | Read | Required by GitHub for GitHub App repository access. |
| Pull requests | Read | Required to subscribe to PR review/comment/thread events and to read PR details, reviews, review comments, commits, and GraphQL review threads. |
| Pull requests | Read and write | Required to subscribe to PR review/comment/thread events, read PR details, reviews, review comments, commits, and GraphQL review threads, and post review-guidance comments on PRs. |
| Members | Read | Reads approver-team membership configured in `repositories.json`. |

PR conversation comments are covered by `Issues: read and write`. The dashboard
does not create inline review comments, submit reviews, or resolve review
threads, so it does not need `Pull requests: write`.
The dashboard does not create inline review comments, submit reviews, or resolve
review threads. It does create PR conversation comments for review guidance,
which requires `Pull requests: read and write`.

Subscribe to events:

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pull-request-dashboard-repo.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ jobs:
owner: open-telemetry
repositories: ${{ inputs.repository }}
permission-issues: write
permission-pull-requests: read
permission-pull-requests: write

- name: Post review guidance
env:
Expand Down