编者按:本文作者Berwin,W3C性能工作组成员,360导航资深前端工程师。《深入浅出Vue.js》作者。
前不久Google开源了一份文档(谷歌工程实践文档),里面包含了他们的代码评审(Code Review)指南,通读之后我发现这份文档非常有价值,所以决定写一篇文章将一些入门的,科普性质的部分翻译并总结成一篇文章帮助大家了解下Google的代码评审。
在Google开源的文档中,有两个内部的专业术语:CL和LGTM:
CL,全称为“Change List”,表示已提交到版本控制或正在进行代码评审的一个独立更改。
我理解和Github上的PR有点类似。
LGTM,表示“Looks Good to Me”,当某个CL被批准后,评审者会说LGTM。
1. 什么是代码评审
代码评审是评审者(某段代码作者之外的人),对某段代码进行检查(审核)的一个过程。
我们可以使用代码评审来维持代码和产品的质量,代码评审的主要目的是确保代码库的整体健康状况会随着时间不断改善。
代码评审的另一个重要功能是,它可以教给开发者一些关于语言,框架,常用的设计原则等知识。
1.1 评审什么
当进行代码评审时,评