问如何扩展代码评审

EN

首先要接受的是，您不能完全自动化代码评审。但是，您可以在不同程度上自动化评审的特定方面，以减轻对审核人员的压力，并让他们专注于无法自动化的事情。使用链接找到(并希望纠正)风格问题。静态分析工具可以突出显示安全性或性能问题、未使用的代码和其他潜在问题。根据工具的不同，开发人员可以在打开代码评审之前在本地运行这些工具，或者可以将它们合并到您的构建过程中，使代码审阅者可以使用报告，甚至导致生成失败。取决于您的技术，可能会有各种解决方案，其中一些是免费的开源软件，另一些则是商业产品。

一旦实现了自动化，您的人就可以专注于系统的体系结构和设计以及代码的可读性等问题。这也将需要文化上的转变。快速检查可能还不够。尽管工具可以找到一些问题，但人类仍然需要花费大量的时间阅读代码并提出问题。我发现在评审中使用更高质量的代码是有帮助的，这无助于改变人们的态度。每个人都应该对团队和组织的代码评审的预期结果或目的保持一致。

大多数情况下，你通过让每个人对他们所写的每个拉请求检查一个拉请求来进行缩放。自动化工具并不能取代人工评审者，但是他们让人类关注更重要的事情。

如果你是一个在评论中经常有更多话要说的评论者，你可以通过评论的级别来衡量。我对我自己的团队(目前有3名其他开发人员)进行了全面的回顾。对于其他团队编写的拉请求，我查看所有的标题，但任何与例行的标题，我忽略了。一些标题值得快速浏览拉请求，而一些快速浏览则需要更深入的回顾。任何我没有深入复习的事情，我都会留给别人去批准。

这个系统让我每天在自己的团队之外有一两次评论。除此之外，是的，偶尔挑剔的事情会被错过，但大多数情况下，你是在判断你的时间花在哪里。

问问自己这个问题--代码审查阶段的意义是什么？

以下是一些可能的答案..。

1. 为了确保代码的一致性--使用CI/CD管道中的自动代码链接器，并拒绝不遵循项目样式指南的代码提交。不需要人类。
2. 审查实施方法-在过程中太晚了。您希望进行设计评审，并进行跟踪，以确保实现遵循设计。代码评审阶段发生在工作结束时，因此修复实现/设计差异的成本很高。
3. 识别代码嗅探-这些是可以被良好的代码分析器识别的。在CI/CD管道中实现这一点，并在未解决这些问题的情况下失败构建。要求将所有豁免作为设计审查的一部分进行审查。
4. 培训/指导练习-当某人在完成任务前开始表现出能力作为最终评审时，这是有一些好处的，但这应该只需要在一段时间内的变化中占一小部分。

简而言之，如果公司愿意花费工程上的精力来集成工具，那么在代码评审中的人工工作量就可以减少。