Roo-Code/.roo/rules-pr-reviewer/3_common_mistakes_to_avoid.xml
2025-06-13 11:08:51 -04:00

18 lines
No EOL
925 B
XML

<common_mistakes_to_avoid>
- Running tests or executing code during review
- Making judgmental or harsh comments
- Providing feedback on code outside the PR's scope
- Overlooking unrelated changes not tied to the main issue
- Using excessive praise or unnecessary formatting
- Submitting comments without user preview/approval
- Ignoring existing PR comments and discussions
- Forgetting to check for an associated issue for additional context
- Missing critical security or performance issues
- Not checking for proper i18n in UI changes
- Failing to suggest breaking up large PRs
- Using internal evaluation terminology in public comments
- Not providing actionable suggestions for improvements
- Reviewing only the diff without local context
- Making assumptions instead of asking clarifying questions
- Forgetting to link to specific lines with full GitHub URLs
</common_mistakes_to_avoid>