V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
jiangbingo
V2EX  ›  程序员

gerrit code review 的一些困惑

  •  
  •   jiangbingo · 303 天前 · 443 次点击
    这是一个创建于 303 天前的主题,其中的信息可能已经有所发展或是发生改变。
    一直在用 gitlab 和 gerrit 做 code review, 发现 review 只能提一些语法和 guideline 方面的 comments. 对代码逻辑很难 review 出来,原因在于 c++ 项目的改动涉及到文件特别多,没有跳转的功能,难以一下子看出所以然。不知各位是如何利用有限时间做 code review ?
    lewinlan
        1
    lewinlan  
       302 天前 via Android
    肉眼主要看思路,能不能跑起来靠测试
    jiangbingo
        2
    jiangbingo  
    OP
       302 天前
    @lewinlan review 的时候一般等 gerrit CI pass 之后。 有时候想看逻辑,于是 checkout 到本地来看。
    wheeler
        3
    wheeler  
       302 天前 via iPhone
    只能 commit 做好切分。尽量单一,改动小。
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   4115 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 19ms · UTC 02:53 · PVG 10:53 · LAX 18:53 · JFK 21:53
    ♥ Do have faith in what you're doing.