这是一个创建于 1388 天前的主题，其中的信息可能已经有所发展或是发生改变。
一直在用 gitlab 和 gerrit 做 code review, 发现 review 只能提一些语法和 guideline 方面的 comments. 对代码逻辑很难 review 出来，原因在于 c++ 项目的改动涉及到文件特别多，没有跳转的功能，难以一下子看出所以然。不知各位是如何利用有限时间做 code review ？
 |
1
lewinlan 2021-02-08 11:00:42 +08:00 via Android
肉眼主要看思路，能不能跑起来靠测试
|
 |
2
jiangbingo OP @lewinlan review 的时候一般等 gerrit CI pass 之后。 有时候想看逻辑，于是 checkout 到本地来看。
|
 |
3
wheeler 2021-02-08 12:39:27 +08:00 via iPhone
只能 commit 做好切分。尽量单一，改动小。
|
Select Language