gerrit code review 的一些困惑

[jiangbingo]

一直在用 gitlab 和 gerrit 做 code review, 发现 review 只能提一些语法和 guideline 方面的 comments. 对代码逻辑很难 review 出来，原因在于 c++ 项目的改动涉及到文件特别多，没有跳转的功能，难以一下子看出所以然。不知各位是如何利用有限时间做 code review ？

[lewinlan]

肉眼主要看思路，能不能跑起来靠测试

[jiangbingo]

@lewinlan review 的时候一般等 gerrit CI pass 之后。 有时候想看逻辑，于是 checkout 到本地来看。

[wheeler]

只能 commit 做好切分。尽量单一，改动小。