|
刚才给新来的consultant作code review,指出几个很严重的performance的问题(小小不然的我都算了),刚开始说他还比较能接受,后来说着说着脸越拉越长,到最后说:我要去赶汽车,我明天问Matt(另外一个snr consultant)怎么做吧... 我基本上忽略他的反应,code review就是质量把关,管你是谁...让我来做,就要符合我的标准,更何况是严重的错误。 自我感觉我算是标准放的很宽的了,原来有同事做frontend code review,能让人起每个browser做兼容性的测试. 大家都是怎么做code review的?照顾同事的面子?还是一丝不苟保证code质量? |