【问题标题】:Merge conflicts in a pull request合并拉取请求中的冲突
【发布时间】:2018-07-26 17:31:19
【问题描述】:

我和一位同事就代码审查进行了一次有趣的讨论。一个说,合并冲突将在批准后解决,另一个说必须解决合并冲突才能获得批准,因此分支已准备好合并。

哪种工作流程在现实世界中效果更好?

【问题讨论】:

  • 他们都更好......所以任何“更好”的定义。

标签: code-organization


【解决方案1】:

TLDR:必须解决合并冲突才能获得批准

首先:如果你对代码 A 没问题,你会用代码 A+B 吗? (其中 A 和 B - 一些提交)。

换句话说:如果您接受了代码审查,那么您应该在下一次代码更改后自动拒绝您的批准。

因此,每个代码更改(或额外提交)都应拒绝批准。

下一步: 如果没有解决冲突,您将无法合并代码。因此,您需要在合并之前更改代码(例如,您必须推送新版本)。此操作将拒绝所有批准,因此:您必须先修复合并冲突,然后才能设置批准

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 2020-12-15
    • 1970-01-01
    • 2020-01-21
    • 1970-01-01
    • 2020-12-28
    • 1970-01-01
    • 2021-09-08
    • 2018-01-30
    相关资源
    最近更新 更多