Skip to content

Commit 25b955c

Browse files
authored
fix: align CodeRabbit and Copilot review configs with other repos (#20)
## Summary - Add `language: en-US` and `chat: auto_reply: true` to `.coderabbit.yaml` to match other repos - Add `drafts: false` to auto_review settings in `.coderabbit.yaml` - Restructure `.github/copilot-instructions.md` with proper H1 header, `## Review Priority` and `## Expectations` sections ## Test plan - [ ] Verify CodeRabbit picks up the language and chat settings on next PR review - [ ] Verify Copilot code review still triggers with restructured instructions <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated code review automation configuration to exclude draft pull requests from automatic reviews and enabled automatic chat reply functionality. * Established comprehensive code review standards documentation with explicit requirements for code quality validation, security practices, and standardized commit conventions. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 23023fb commit 25b955c

File tree

2 files changed

+16
-1
lines changed

2 files changed

+16
-1
lines changed

.coderabbit.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
1+
language: en-US
12
reviews:
23
auto_review:
34
enabled: true
5+
drafts: false
46
path_instructions:
57
- path: "scripts/**/*.sh"
68
instructions: |
@@ -33,3 +35,5 @@ reviews:
3335
- "!**/*.png"
3436
- "!**/*.gif"
3537
- "!**/*.jpg"
38+
chat:
39+
auto_reply: true

.github/copilot-instructions.md

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
1-
# Review Priorities
1+
# GitHub Copilot Code Review Instructions
2+
3+
## Review Priority
24

35
1. Shell script quality: shellcheck and shellharden compliance, proper
46
quoting, error handling (set -euo pipefail), no hardcoded tokens
@@ -12,3 +14,12 @@
1214
permissions, no script injection
1315
6. Markdown quality: 120 char line limit, fenced code blocks, accurate
1416
documentation
17+
18+
## Expectations
19+
20+
- All shell scripts must pass shellcheck and shellharden without
21+
suppressions.
22+
- Variables must be quoted (`"$VAR"`), braces only when needed.
23+
- No hardcoded credentials, account IDs, or tokens in any file.
24+
- Conventional commits required for all changes.
25+
- Fix lint violations directly instead of adding ignore comments.

0 commit comments

Comments
 (0)