Code Review是一个很好的process，大家互相点评code，这样可以提高code quality。
是的，没错。但是如果大家不互相点评code，每个PR都是点一下approve，是不是就说明code review没做好，是不是就说明code的quality有问题呢。
一个很厉害的team，每个人写的code大家互相都挑不出刺来，那么自然就没有那么多code review comments，自然就是互相点approve，此时code review没有comments反而表明了quality足够好所以大家都没有comments。
很多时候我们当领导的都有思维惯性，忘记了最终目标，一旦comments少了就认为code review没做好，然后就想当然的认为code quality会有问题。但我们的最终目标是什么，是code quality到底好不好，而不是code review这个process做的好不好。事实上对于成熟项目大家都是follow现有框架开发，不会差别到哪里去，有问题的PR毕竟是少数，10个PR里能有1-2个PR有建设性的comments已经很不错了。我们老是盯着process看有没有做到位，但是忘记了最终的目标。而且我们也忘记了process和最终目标间没有那么强的必然的因果联系，特别是在编码这块，我之前文章说了我们太过于习惯用秩序的思维，因果关系来看这个问题，然而编码本身是混沌的。这就是我们by default的default错了啊。
那怎么办呢，我觉得还是陆奇说的好，领导要能做下面2级的人的事情。如果领导能亲自看提交的code，那么他就能知道这code行不行，那么就不会只认code review comments多还是少了。