—— Code Review 是一种文化,而非制度。
目标
- 提高代码质量和可读性
- 共享知识与编程技巧
- 给予有意义的反馈帮助小伙伴们成长
注:因为“旁观者清,当局者迷”,所以需要伙伴的帮助来发现个人的不足。
时机
代码待合并且完成以下步骤:
原则
- 代码逻辑是否简单明了?
- 代码是否符合公司规范?
- 单次 Review 时间建议少于 30 分,不超过 60 分(可分批 Review)
- 单次查阅建议少于 400 行,不超过 1000 行
- 确保代码逻辑正确,安全可靠、可读性强(而非个人偏好)
- 不要在 Review 中讨论需求(Review 是 Review,Planning 是 Planning)
- 面对面给予正向反馈(文明用语,尊重彼此,保持互相学习的心态)
注:没有完美的代码,不要纠结。
关注清单
根据团队需要定制规范