Prefer inline Claude review comments (#10970)

WISOTT
This commit is contained in:
Stephan Behnke
2026-07-08 09:48:38 -07:00
committed by GitHub
parent 76bdf3bac1
commit f3c440d827
2 changed files with 3 additions and 10 deletions

View File

@@ -6,13 +6,4 @@ allowed-tools: Read, Grep, Glob
# Code Review Skill
Apply these review patterns when reviewing code or suggesting changes.
@.github/copilot-instructions.md
## Review Output Format
When reviewing, organize feedback by priority:
1. **Must fix**: Issues that will cause bugs or break conventions
2. **Should fix**: Redundant code, naming issues, test correctness
3. **Nit**: Style preferences, minor improvements

View File

@@ -164,8 +164,10 @@ jobs:
Use `.claude/skills/review/SKILL.md` and apply it as the review skill.
Please review this pull request.
The PR branch is already checked out in the current working directory.
Use `gh pr comment` for top-level feedback.
Prefer inline comments for all actionable findings.
Use `mcp__github_inline_comment__create_inline_comment` with `confirmed: true` to highlight specific code issues.
Only use `gh pr comment` for a brief summary if there are no inline findings or if a finding cannot be attached to a specific changed line.
Do not duplicate inline findings in a top-level comment.
Only post GitHub comments - don't submit review text as messages.
claude_args: |
--allowedTools "Read,Grep,Glob,mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*)"