首页/开发/代码质量与评审速查/合并前检查(pre-merge checklist)
代码评审64 条

合并前检查(pre-merge checklist)

点合并前快速核对:CI 全绿、批准齐全、讨论已解决、目标分支正确、无冲突与调试残留;赶时间时这些项更不能跳过。

合并前检查

合并前检查
1. CI 全部通过(含 lint、测试)
2. 所需批准已齐(含 Code Owner)
3. 所有对话已解决(Resolve conversation)
4. 目标分支与基准正确,无冲突
5. 版本号、变更日志、开关状态就绪
6. 合并方式与提交信息符合团队规范
高风险改动临近周末合并:出事只有值班同学兜底,时机也是评审的一部分。

易错点

  • 看都不看直接点绿色按钮:CI 未覆盖的变更照样出问题
  • 合并后忘记清理分支与查监控:环境越堆越乱

数据来源与参考

  • GitHub Docs · Merging a pull request(合并前状态)
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌