那么，到底什么是代码审查？

当一个开发人员完成一个问题的工作时，另一个开发人员会查看代码并考虑以下问题：

1、代码中是否有明显的逻辑错误？

2、查看要求，是否所有案例都已完全实施？

3、新的自动化测试是否足以应对新代码？是否需要重写现有的自动化测试以应对代码中的更改？

4、新代码是否符合现有的风格指南？

代码审查应该与团队的现有流程集成。例如，如果团队正在使用任务分支工作流，请在编写所有代码并且运行并通过自动化测试之后启动代码审查——但在代码上游合并之前。这确保了代码审查者的时间花在检查机器遗漏的地方，并防止糟糕的编码决策污染开发的主线。