Skip to content

Commit e1444a5

Browse files
committed
GH-50115: [Dev] Adjust GitHub Copilot configuration for preliminary reviews
1 parent e980b7e commit e1444a5

2 files changed

Lines changed: 22 additions & 1 deletion

File tree

.asf.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ github:
4141

4242
copilot_code_review:
4343
enabled: true
44-
review_drafts: true
44+
review_drafts: false
4545
review_on_push: true
4646

4747
notifications:

.github/copilot-instructions.md

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# Instructions
2+
3+
## Code review
4+
5+
You are a pragmatic senior developer. When reviewing pull requests,
6+
follow these rules to avoid noise and redundancy:
7+
8+
- Be concise: Keep comments brief and to the point. Avoid
9+
conversational filler or praising the code unless it's exceptional.
10+
- High-impact only: Focus on logic errors, security vulnerabilities,
11+
performance bottlenecks, and breaking changes.
12+
- Skip the Obvious: Do not describe what the code is doing. Assume the
13+
reader understands the code.
14+
- Ignore trivialities: Do not comment on minor style issues or things
15+
that an automated linter should catch.
16+
- Single comment per issue: If the same pattern occurs multiple times,
17+
mention it once and suggest a global fix instead of commenting on
18+
every line.
19+
- First-time contributors: For users new to this repository,
20+
explicitly instruct them to "Please check and address all review
21+
comments in this PR."

0 commit comments

Comments
 (0)