技术科普

代码评审跟随太阳:让PR在睡眠中前进

一个 pull request 提交后,最怕的是停在“等待评审”状态。若评审者都在另一个时区睡觉,几小时就这样过去了。跨时区团队于是设计出“跟随太阳”的评审流程。

跟随太阳怎么运作

当亚太团队下班前提交代码,欧洲团队上班后接手评审;欧洲团队下班前处理完,美洲团队继续。这样一轮评审可以在二十四小时内完成,而不是所有人都挤在同一个窗口。

让交接更顺滑

  • 提交时写清改动目的、影响范围和测试结果。
  • 标注需要重点看的部分,减少评审者的理解成本。
  • 使用自动化检查处理格式、 lint 和基础测试。
  • 约定评审时限,例如一个工作日内给出第一轮意见。

度量什么

可以统计从提交到首次评审、从首次评审到合并的时间,找出瓶颈。若等待时间集中在某个时区,说明评审人力分布不均。也可以看 PR 大小:过大的改动需要更长时间评审,跨时区时更容易积压。

别把时区当成拖延的理由

跟随太阳不是把工作无限接力。每个环节都应有明确的负责人和截止时间。若评审意见需要多轮往返,应尽早安排同步讨论,而不是在评论里来回几天。

好的评审流程,让代码在人不醒着的时候也能向前走一小步。