V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
b00tyhunt3r
V2EX  ›  程序员

merge 遇到 conflict 应该由哪边 resolve ?

  •  
  •   b00tyhunt3r · 88 天前 · 801 次点击
    这是一个创建于 88 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

    10 条回复    2021-09-04 11:36:35 +08:00
    RRRoger
        1
    RRRoger   88 天前
    从目标分支新建一个分支 A, 用自己分支往 A merge, 解决冲突,用 A 往目标分支 merge
    b00tyhunt3r
        2
    b00tyhunt3r   88 天前
    @RRRoger
    看描述你这个 merge 方(审核方) 和 PR 的提交方是同一个人吗?
    MatDK
        3
    MatDK   88 天前
    一般是提交方 创建一个分支解决 conflict,然后重新 PR 。让 merge 方 review 。
    当然有条件 2 边同时在场解决 conflict 也可以
    b00tyhunt3r
        4
    b00tyhunt3r   88 天前
    @MatDK
    不知道是否存在这个可能
    代码冲突的地方提交方不知道怎么改,或者没有权利改?
    MatDK
        5
    MatDK   88 天前
    @b00tyhunt3r 那就只能把 merge 方找来了,偶尔也确实这样没办法
    kop1989
        6
    kop1989   88 天前
    同意 3L 意见。

    btw:分支合并、冲突、解决冲突、发起合并请求( PR/MR )。这不难打吧,也并没有歧义。
    HankAviator
        7
    HankAviator   87 天前 via Android
    @kop1989 v2 上混英语的多了去了,你这样建议累死都没人听😬
    谁引起谁处理,光拉不埋可不行
    neoblackcap
        8
    neoblackcap   87 天前
    你提 PR 合并到主线,你的提交出现了冲突,当然是你要解决。
    b00tyhunt3r
        9
    b00tyhunt3r   87 天前
    @neoblackcap
    理是这个理

    但是作为提交方有时候摸不准冲突的部分能不能动
    merge 方直接改的话不需要扯皮效率会高很多
    yuguorui96
        10
    yuguorui96   87 天前
    @b00tyhunt3r 那你修改代码前要明确改动,与 merge 方同步再做他们允许的逻辑修改,而不是先自己闷头改。
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   3031 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 32ms · UTC 10:32 · PVG 18:32 · LAX 02:32 · JFK 05:32
    ♥ Do have faith in what you're doing.