本节中介绍了基于长期经验总结的代码评审最佳实践。所有内容都在同一个章节里,但被分为了好多子章节。虽然你不必通读一遍,但认真看一遍对你自己和整个团队都会有很大的益处。
- 尽快评审,不要拖延
- 大的变更 如果有人给你提交了一个非常大的代码评审,你也不确定你有时间看,你最好建议开发者把MR拆分成几个小的部分,分多次代码评审,而不是一次性全部提交上来,而且拆分也有利于评审者。 作为评审者,你的目标之一是在不牺牲代码质量的前提下,不阻碍开发者的进程或者尽可能让他们向前推进
- LGTM 通过一些术语向开发者表示允许合并