【发布时间】:2018-07-26 17:31:19
【问题描述】:
我和一位同事就代码审查进行了一次有趣的讨论。一个说,合并冲突将在批准后解决,另一个说必须解决合并冲突才能获得批准,因此分支已准备好合并。
哪种工作流程在现实世界中效果更好?
【问题讨论】:
-
他们都更好......所以任何“更好”的定义。
我和一位同事就代码审查进行了一次有趣的讨论。一个说,合并冲突将在批准后解决,另一个说必须解决合并冲突才能获得批准,因此分支已准备好合并。
哪种工作流程在现实世界中效果更好?
【问题讨论】:
TLDR:必须解决合并冲突才能获得批准
首先:如果你对代码 A 没问题,你会用代码 A+B 吗? (其中 A 和 B - 一些提交)。
换句话说:如果您接受了代码审查,那么您应该在下一次代码更改后自动拒绝您的批准。
因此,每个代码更改(或额外提交)都应拒绝批准。
下一步: 如果没有解决冲突,您将无法合并代码。因此,您需要在合并之前更改代码(例如,您必须推送新版本)。此操作将拒绝所有批准,因此:您必须先修复合并冲突,然后才能设置批准
【讨论】: