General (+5)
General
- Code compiles/builds without errors
- No merge conflicts
- Branch is up to date with target
- Commit messages are clear
Functionality
- Requirements are met
- Edge cases handled
- Error handling is appropriate
- No regression in existing functionality
Code Quality
- Follows project coding standards
- No code smells or anti-patterns
- Appropriate comments (not excessive)
- No dead code or debug statements
Security
- No hardcoded secrets
- Input validation present
- Output encoding where needed
- Proper error messages (no info leak)
Testing
- New code has tests
- All tests pass
- Test coverage adequate
- Tests are meaningful
Documentation
- README updated if needed
- API docs updated if applicable
- Breaking changes documented