首页/开发/代码质量与评审速查/小 CL 原则:一次评审一个自包含改动
代码评审64 条

小 CL 原则:一次评审一个自包含改动

Google 评审文档建议把 CL 控制在小规模(约 100 行为参考),每个 CL 是单一自包含改动,便于评审、合并与回滚。

参考规模约 100 行(文档参考值,按情况调整)
形态单一、自包含、可独立理解
  • 小改动更容易被认真评审,缺陷更早暴露
  • 合并更快,减少与其他分支的冲突与返工
  • 回滚半径小,线上问题影响面可控
  • 重构与功能改动分开:各自独立成 CL
Google 文档同时提醒:拆分的前提是每个 CL 仍是一个完整、可独立合入的改动。

易错点

  • 拆得过碎:每个 CL 都不完整,依赖链拉长反而拖慢进度
  • 为控制行数把无关小修混进同一 CL,多个主题互相纠缠

数据来源与参考

  • Google Engineering Practices · Small CLs
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌