my-git
Code Review 清单
意图
The purpose of the change is clear
The PR solves one problem
Unrelated changes are split out
行为变化
Existing behavior is preserved unless intentionally changed
Edge cases are considered
Error handling is explicit
测试
Tests cover the changed behavior
Tests are meaningful
Verification steps are documented
可维护性
Code follows existing style
Naming is clear
No unnecessary abstraction is introduced
风险
Security impact is considered
Rollback path is clear
Operational impact is documented