• 请不要在回答技术问题时复制粘贴 AI 生成的内容
bingoAI
0.01D
V2EX  ›  程序员

gerrit code review 的一些困惑

  •  
  •   bingoAI · Feb 8, 2021 · 1428 views
    This topic created in 1942 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
    肉眼主要看思路,能不能跑起来靠测试
    bingoAI
        2
    bingoAI  
    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   ·   4535 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 61ms · UTC 01:03 · PVG 09:03 · LAX 18:03 · JFK 21:03
    ♥ Do have faith in what you're doing.