一直在用 gitlab 和 gerrit 做 code review, 发现 review 只能提一些语法和 guideline 方面的 comments. 对代码逻辑很难 review 出来,原因在于 c++ 项目的改动涉及到文件特别多,没有跳转的功能,难以一下子看出所以然。不知各位是如何利用有限时间做 code review ?
1
lewinlan Feb 8, 2021 via Android
肉眼主要看思路,能不能跑起来靠测试
|
3
wheeler Feb 8, 2021 via iPhone
只能 commit 做好切分。尽量单一,改动小。
|