merge 遇到 conflict 应该由哪边 resolve ?

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 1177 天前的主题，其中的信息可能已经有所发展或是发生改变。

如果提交 PR 的时候遇到 conflict, 你们公司的流程是提交方自己 resolve 好，再把解决好不冲突的 PR 重新提交

还是 merge 方在 merge 的时候当场解决 conflict，改好直接完成 merge ？

10 条回复 • 2021-09-04 11:36:35 +08:00

RRRoger

2021-09-03 15:40:07 +08:00

从目标分支新建一个分支 A，用自己分支往 A merge，解决冲突，用 A 往目标分支 merge

b00tyhunt3r

2021-09-03 15:44:21 +08:00

@RRRoger
看描述你这个 merge 方（审核方）和 PR 的提交方是同一个人吗？

MatDK

2021-09-03 15:50:40 +08:00

一般是提交方创建一个分支解决 conflict，然后重新 PR 。让 merge 方 review 。
当然有条件 2 边同时在场解决 conflict 也可以

b00tyhunt3r

2021-09-03 15:57:33 +08:00

@MatDK
不知道是否存在这个可能
代码冲突的地方提交方不知道怎么改，或者没有权利改？

MatDK

2021-09-03 16:01:34 +08:00

@b00tyhunt3r 那就只能把 merge 方找来了，偶尔也确实这样没办法

kop1989

2021-09-03 16:02:09 +08:00

同意 3L 意见。

btw：分支合并、冲突、解决冲突、发起合并请求（ PR/MR ）。这不难打吧，也并没有歧义。

HankAviator

2021-09-03 21:23:38 +08:00 via Android

@kop1989 v2 上混英语的多了去了，你这样建议累死都没人听😬
谁引起谁处理，光拉不埋可不行

neoblackcap

2021-09-04 02:38:36 +08:00

你提 PR 合并到主线，你的提交出现了冲突，当然是你要解决。

b00tyhunt3r

2021-09-04 08:30:37 +08:00

@neoblackcap
理是这个理

但是作为提交方有时候摸不准冲突的部分能不能动
merge 方直接改的话不需要扯皮效率会高很多

yuguorui96

2021-09-04 11:36:35 +08:00

@b00tyhunt3r 那你修改代码前要明确改动，与 merge 方同步再做他们允许的逻辑修改，而不是先自己闷头改。