Code Review¶
即使 code review 会花掉哪怕十倍于你自己写的时间和精力,但它会让人明白代码应该怎么写,从长远来看,这其实是在一定程度上 “复制” 你的生产力。你不能什么都自己写。尤其是你慢慢开始带项目、带新人。 写代码是一个学习过程,怎么做一个好的代码审核人更是一个学习和成长的过程
代码格式方面。
代码可读性方面。
帮代码作者想想他/她有没有漏掉任何 corner case
Error handling和问题定位
❤code review的几条规则:
1. 每个 PR 审查至少需要 2 个同组开发者的批准,管理者的审批不统计。
2. 每个 PR 必须有一个好的描述,通过阅读描述,审核者能够理解代码的目的是什么。即使有一个 Jira ticket 或需求页面,也必须满足此要求。
3. PR 必须有足够的单元测试和集成测试覆盖。
4. 如果 PR 是一个 bug 修复,它必须包含一个测试,如果这个 bug 被还原,这个测试就会报错。