Gerrit Code Review工具

Code Review 主要有以下目的：发现错误：人都会不可避免的出现一些纰漏，而这些纰漏在另一个人眼中也许显而易见。健壮性检查：代码是否健壮，是否有潜在安全、性能风险。代码是否可以回滚。质量保证：在一般情况下，新提交的代码一定需要写测试，测试不只可以保证你的提交符合预期，还可以在后人改你的代码时有一层保障。同时，MR 阶段也有机器人自动检查当前分支的测试覆盖率是否低于主分支，当低于主分支时会标红警示，但不会禁止 merge。统一风格：对于整个团队来说，代码风格的统一很重要。风格统一除了人 Review，我们也引入了静态代码检查，不符合团队风格的代码，是无法通过 CI 的。完善注释：包括 commit message、代码中复杂实现是否有解释性的注释、紧急 hack 是否明确标注等。互相学习一般来讲，
一个 MR (merge request)不宜过大。如果是大的 feature 改动，可以分成多个小 MR 分次提交。

Gerrit Code Review工具

没有这个工具的时候写代码…
写代码→add→commit→push→远程仓库
有这个工具的时候写代码…
写代码→add→commit→push→Gerrit→远程仓库
代码被提交到Gerrit上的时候，同事们就可以来细品你写的代码了，有问题就修改，没问题才提交到远程库中