我现在公司，这点氛围很好。责任感强的人
多了在一起，工作就觉得有意思些
cr，核心是发现瑕疵，而不是挑刺，严禁对人批评。
我们必须要review，但可以是任何人approve，也可以任何人题review意见。

我review得稍慢些，没那么积极，leader和CTO，是重构最多的人了，review也很积极。

核心还是人的因素，前公司，氛围不好，也有review，总在吹毛求疵是不对的。一个pr，只要没有bug，对原代码质量有提高，对功能改进就应该approve，那些拼写错误，指出来就行。




【 在 dpblue 的大作中提到: 】
: 问题：如何给同事做code review同时又不会惹人反感？
: 就像你前面说的，你觉得我说的两个做法都算错，但你楼上认为code review不应该管，也就是说如果是你review他的代码，给了意见说不行，那他一定很不满，心想：”明明不应该管的东西，干嘛非要提，害我没法merge“，这个怎么破？
:
--
FROM 117.136.0.*