Reviewing code you didn't write
9 hours ago
- 审查代码是一项与编写代码不同的技能，重点在于发现决定工作是否应该发布的关键问题。
- 通过签出分支、阅读测试以及从工单或设计说明中理解意图，来重构变更的上下文。
- 首先审查核心路径：在深入细节之前评估方法并检查重复。
- 按优先级排序审查意见：方法、正确性、失败、安全性和证据；仅对关键问题阻止合并，并使用'nit:'表示偏好。
- 在审查AI生成的代码时，要对虚构的API、不必要的抽象和过于完整的测试套件保持怀疑；根据任务和项目规则进行验证。
- 通过学习已合并的PR以及就自己的审查意见向其他工程师寻求反馈，来提高审查技能。