代码审查惯例

  • 代码审查完成之前该代码通过拉取请求合并到主分支中。
  • 工程师的直接经理应添加到“每个拉动请求”中。
  • 代码审阅由审阅者​​自己完成。
  • 必须先获得所有审稿人的批准,然后才能批准请求。


代码审查动机

  • 提交者有动力编写更好的代码,以了解是否有人会看它。
  • 学习和分享有关如何编写更好的代码的知识。
  • 设计和编码风格上的一致性。
  • 尽早发现错误,反模式和设计缺陷。