代码评审检查清单与提交合并规范落地.docx

代码评审检查清单与提交合并规范落实

评审范围界定与准入前置条件

××团队划了一条硬线：改动触及约3个文件、涉及1个核心模块或新增1张数据表时必须走评审，纯格式化与注释微调可免审。准入门槛把日均评审量压到每人约5个合并请求，免得评审队列堆到30个却没人细看。

评审前先自检一遍：本地编译通过、单元测试跑绿、静态扫描无高危项。缺任一项当轮退回，约10分钟内就能判定，不占用别人的时段。返工记录显示，跳过自检的请求平均多花约1.5个人日才合并。

范围模糊是评审拖期的首因。××要求每个请求标注受影响接口数与预计回滚耗时，回滚超过30分钟的按高风险单独排会，评审人提前2小时到场读码。

改动规模

评审方式

参