Skip to content

Commit 4decac8

Browse files
[gh-aw] Android PR Reviewer: restrict to COMMENT events only (#11762)
Remove REQUEST_CHANGES from allowed-events so the reviewer workflow can only post comments, not block PRs by requesting changes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 824e81e commit 4decac8

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/android-reviewer.lock.yml

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/android-reviewer.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ safe-outputs:
3939
max: 50
4040
submit-pull-request-review:
4141
max: 1
42-
allowed-events: [COMMENT, REQUEST_CHANGES]
42+
allowed-events: [COMMENT]
4343
---
4444

4545
# Android PR Reviewer
@@ -65,6 +65,6 @@ A maintainer commented `/review` on this pull request. Perform a thorough code r
6565
- If the same issue appears many times, flag it once listing all affected files.
6666
- Don't flag what CI catches (compiler errors, linter issues).
6767
- Avoid false positives — verify concerns given the full file context and project configuration (TargetFramework, references, available APIs).
68-
- **Never submit an APPROVE event.** Use COMMENT for clean PRs and REQUEST_CHANGES when issues are found.
68+
- **Always submit the review as a COMMENT event.** Never APPROVE or REQUEST_CHANGES — surface issues in the comment body instead.
6969
- Prioritize: bugs > safety > performance > missing tests > duplication > consistency > documentation.
7070
- **Post suggestions as inline comments, not just in the summary.** If a suggestion can't be posted inline, omit it.

0 commit comments

Comments
 (0)