主页

索引

模块索引

搜索页面

Code Review

即使 code review 会花掉哪怕十倍于你自己写的时间和精力,但它会让人明白代码应该怎么写,从长远来看,这其实是在一定程度上 “复制” 你的生产力。你不能什么都自己写。尤其是你慢慢开始带项目、带新人。 写代码是一个学习过程,怎么做一个好的代码审核人更是一个学习和成长的过程

  1. 代码格式方面。

  2. 代码可读性方面。

  3. 帮代码作者想想他/她有没有漏掉任何 corner case

  4. Error handling和问题定位

❤code review的几条规则:

1. 每个 PR 审查至少需要 2 个同组开发者的批准,管理者的审批不统计。
2. 每个 PR 必须有一个好的描述,通过阅读描述,审核者能够理解代码的目的是什么。即使有一个 Jira ticket 或需求页面,也必须满足此要求。
3. PR 必须有足够的单元测试和集成测试覆盖。
4. 如果 PR 是一个 bug 修复,它必须包含一个测试,如果这个 bug 被还原,这个测试就会报错。

主页

索引

模块索引

搜索页面