V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member  Sign In
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
jiangbingo
0.01D
V2EX  ›  程序员

gerrit code review 的一些困惑

  •  
  •   jiangbingo · Feb 8, 2021 · 1348 views
    This topic created in 1904 days ago, the information mentioned may be changed or developed.
    一直在用 gitlab 和 gerrit 做 code review, 发现 review 只能提一些语法和 guideline 方面的 comments. 对代码逻辑很难 review 出来,原因在于 c++ 项目的改动涉及到文件特别多,没有跳转的功能,难以一下子看出所以然。不知各位是如何利用有限时间做 code review ?
    lewinlan
        1
    lewinlan  
       Feb 8, 2021 via Android
    肉眼主要看思路,能不能跑起来靠测试
    jiangbingo
        2
    jiangbingo  
    OP
       Feb 8, 2021
    @lewinlan review 的时候一般等 gerrit CI pass 之后。 有时候想看逻辑,于是 checkout 到本地来看。
    wheeler
        3
    wheeler  
       Feb 8, 2021 via iPhone
    只能 commit 做好切分。尽量单一,改动小。
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   6076 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 2616ms · UTC 02:36 · PVG 10:36 · LAX 19:36 · JFK 22:36
    ♥ Do have faith in what you're doing.