关于这个问题,一直有争论。
在check-in之前进行review的好处是保证repository中代码的整洁,缺点是增加了代码分享的延迟,延长了集成测试的周期。
关于check-in之后进行review,参加此文:《Check In Early, Check In Often》
在stackoverflow对此也有讨论:
其中提出了使用Git作为版本控制工具的方案。
本人认为,选择之前还是之后要看开发组成员的整体开发水平,如果水平都比较高,则可采取先checkin再审查,以提高效率;
如果整体水平不高,还是先review再checkin比较可取,以保证代码质量。