目标
大型变更应被有效评审而不牺牲质量。本工作流先判断变更能否拆为可独立评审的部分,若不能则按结构、逻辑、测试等关注点分轮评审,要求干净的提交历史以便逐提交阅读,并确认高风险区域得到充分关注而非平均分散。
步骤要点
- 判断变更能否拆分。
- 不能拆分时按关注点分轮评审。
- 要求干净提交历史以便逐提交阅读。
- 确认高风险区域得到充分关注。
# review commit by commit
git log --oneline origin/main..HEAD
验证
按顺序执行上述步骤后,逐项确认每个步骤的预期状态均已满足;只有全部预期状态成立,才认为本流程完成。若某一步未达到预期状态,应先按排错提示排查,不要跳过继续,以免掩盖问题。
排错提示
- 对大型 PR 走过场会隐藏缺陷。
- 逐行评审巨型 diff 会让评审者疲惫。
- 历史混乱会掩盖意图。
适用边界
- 机械可拆的部分应拆分。
- 内聚变更可整体评审。