- 信号:大量本可后续处理的意见被标为阻塞
- 信号:反复要求再改改但不说明标准
- 对策:回到评审标准——改动整体变好即可放行
- 对策:把个人偏好与必须项分离,团队规则公开
意见分级:阻塞还是后续
意见分级:阻塞还是后续
1. 影响正确性、安全或可维护性吗?否 → 标 nit / non-blocking,不挡合并。
2. 有团队规则或事实依据吗?只有偏好 → 降级为非阻塞并注明理由。
3. 定为阻塞:给出通过标准——改到什么程度可以合并。
阻塞项只保留少数几条,其余转入后续任务或单独讨论。过度把关场景与校准
| 场景 | 校准动作 |
|---|---|
| 要求改成个人习惯的等价写法 | 风格交给格式化工具与团队规范,聚焦可验证问题 |
| 反复要求再改改,不说标准 | 给出明确通过条件:问题位置与达标要求 |
| 小瑕疵一律列为阻塞项 | 先评估是否影响正确性;不影响则标非阻塞放行 |
评审的目标是让代码前进,而不是证明评审者更聪明。