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

gerrit code review 的一些困惑

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