谷歌如何进行代码审查
内容提要
Google有两个内部代码审查工具：Critique和Gerrit。Critique允许作者突出显示不同组的更改列表，而Gerrit提供测试覆盖度指标和更详细的更改列表状态。在提交更改之前，需要通过三个强制性审批级别：LGTM、代码所有者和可读性。Gerrit和Critique具有跟踪需要哪些操作来解除特定更改阻塞的功能。Google的代码审查流程全面而周到，整个审查过程的中位延迟时间不到4小时。
延伸解读
工具差异与适用场景
Critique 和 Gerrit 是 Google 内部并行的两个代码审查工具。Critique 主要供大多数工程师使用，其仪表板支持多区域展示，便于作者突出显示不同组的更改列表；Gerrit 则开源并继续用于面向公众的项目，如 Chromium 和 Android。Gerrit 的仪表板提供更详细的更改状态和测试覆盖度指标。两者都旨在提升审查效率，但适用场景不同。
三级审批与可读性审查
在 Google 提交更改前必须通过三级审批：LGTM、代码所有者和可读性。LGTM 确认核心业务逻辑正确，任何人均可给出；代码所有者需批准特定文件的更改；可读性审查是 Google 特有，确保代码符合语言标准和最佳实践。若作者本人是代码所有者或可读性专家，则可自动通过相应级别。这一流程旨在维持大规模代码库的一致性。
注意力集与审查效率
为跟踪谁需要采取行动以解除更改阻塞，Gerrit 和 Critique 引入了“注意力集”概念。通过灰色箭头或加粗名字标记当前需要行动的人，并提示原因。审查者也可标记“不是我的回合”以移除自己。这种机制明确了责任，减少了等待时间。结合仪表板和快捷键，Google 的代码审查中位延迟时间不到 4 小时，远快于行业平均水平。
审查文化与新成员适应
Google 的代码审查文化强调彻底和深思熟虑，这体现在自定义工具和对审查的重视上。新加入的工程师通常需要适应期，他们的第一个更改往往需要数天和多次反馈才能提交。这种文化虽然可能显得繁琐，但有助于知识共享和代码质量提升。审查速度得益于工具和文化双重因素，使得 Google 在保持高质量的同时实现快速审查。
Q&A
谷歌使用哪些工具进行代码审查？
谷歌使用两个内部代码审查工具：Critique和Gerrit。
在谷歌，代码提交前需要经过哪些审批？
在谷歌，代码提交前需要经过三个强制性审批：LGTM、代码所有者和可读性审查。
Critique和Gerrit的仪表板有什么不同？
Critique的仪表板允许作者突出显示不同组的更改，而Gerrit提供更详细的更改状态和测试覆盖度指标。
谷歌的代码审查流程的中位延迟时间是多少？
谷歌的代码审查流程的中位延迟时间不到4小时。
可读性审查在谷歌的代码审查中有什么特别之处？
可读性审查是谷歌特有的，旨在确保代码符合语言标准和最佳实践。
谷歌的代码审查文化与其他公司有什么不同？
谷歌的代码审查文化强调更彻底和深思熟虑的审查过程，通常比行业标准更为严格。
