我和我的同事对代码审查进行了有趣的讨论。一个说,合并冲突将在批准后解决,另一个说必须解决合并冲突才能获得批准,因此分支机构已准备好进行合并。
哪种工作流程在现实世界中能更好地工作?
答案 0 :(得分:1)
TLDR:必须解决合并冲突才能获得批准
首先:如果您对代码A没问题,您是否对代码A + B满意? (其中A和B-一组提交)。
换句话说:如果您接受了代码审查,那么您应该在下一次代码更改后自动拒绝批准。
因此,每次代码更改(或其他提交)都应拒绝批准。
下一步: 没有冲突解决,您将无法合并代码。因此,您需要在合并之前更改代码(例如,您必须推送新版本)。该操作将拒绝所有批准,因此:必须先设置合并冲突,然后才能设置批准