merge 遇到 conflict 应该由哪边 resolve ?

2021-09-03 15:34:08 +08:00
 b00tyhunt3r

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

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

1590 次点击
所在节点    程序员
10 条回复
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
@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 方同步再做他们允许的逻辑修改,而不是先自己闷头改。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/799692

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX