code-review

作为reviewer，一定要读懂diff；所有被accept的diff必须是在读懂的前提下。做reviewer的人要有“将来如果这些代码线上出问题，我要帮助support”的心理准备。

code review 应该被每个engineer当做工作的重要一部分。做Performance Review的时候应该把帮助做过的code review考虑，for both employee & manager.

应当在24小时内给回复，这应当变成共识。

感觉有问题的代码，一定要在相应的行上做出评论 （inline comments)，以让作者明白问题所在。

尽可能把对修改的所有意见一次性给出，减少来来回回的次数。比较复杂的建议reviewer主动找author来进行线下沟通，达成一致。

一般的diff，来回次数不宜超过3次；如果超过3次，想想看，是不是diff 太大，太复杂了。

check-in

与master做一次merge update，确保没有问题

run一次code change涉及到的所有test cases（包括新增的和涉及到的test cases)

----菏泽软件开发----