| 文档 | The Standard of Code Review(Google Engineering Practices) |
|---|---|
| 决策 | 批准让代码库整体变好的改动 |
| 阻止条件 | 改动让代码质量明确下降 |
- 评审目的是随时间保持代码库健康,而不是追求完美
- 没有完美的代码,只有更好的代码;'还可以更好'不是阻塞理由
- 改动确实改善系统时:评审者不应为小瑕疵挡下它
- 反之,若改动让质量下降(例如删掉测试、引入明显耦合),应明确指出并阻止
这条标准同时约束双方:评审者别为小事阻塞,作者也别把未成型的代码丢给评审。
Google 工程实践文档给出的评审决策标准:只要改动让代码库整体变好就应放行,不要求完美;当改动让代码明确劣化时才阻止。
| 文档 | The Standard of Code Review(Google Engineering Practices) |
|---|---|
| 决策 | 批准让代码库整体变好的改动 |
| 阻止条件 | 改动让代码质量明确下降 |