首页/开发/代码质量与评审速查/多轮评审:每轮都要收敛
代码评审64 条

多轮评审:每轮都要收敛

评审多轮是常态,但要保证每轮收敛:作者处理完阻塞项再请求复核,评审者聚焦增量变化,避免每轮重新打开已决问题。

  • 作者:先把阻塞项全部处理,再点重新评审
  • 评审者:复核时先看新增 diff,再抽查关联结果
  • 已决问题不重开:除非有新事实,否则不翻旧账
  • 轮次失控(多轮仍反复):升级到当面沟通

每轮复核的固定动作

每轮复核的固定动作
作者:逐条回复阻塞项——已修复(贴提交)、说明理由、或另开任务;
阻塞项清零后再点重新评审(Re-request review)。
评审者:只看本轮新增 diff,先确认上轮阻塞项闭环,再找新问题。
新发现开新线索;已决结论只在新事实下重开。
两轮仍拉扯:转同步沟通(语音或当面),结论回写 PR。

轮次状态与处理

轮次状态处理动作
一两轮内闭环正常节奏:高频意见沉淀进自检清单,减少下轮往返
三轮以上仍反复先查描述与标准:补说明、明确通过条件
反复推倒重来停文字往复,转当面或语音对齐方案后重审
收敛的评审把讨论推向结论;发散式评审把结论越推越远。

易错点

  • 每轮全量重读:评审者疲劳,后期改动反而没人细看
  • 把推倒重写当作回应:评审者需重新建立上下文

数据来源与参考

  • Google Engineering Practices · The CL Author's Guide(处理多轮意见)
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌