• 请不要在回答技术问题时复制粘贴 AI 生成的内容
b00tyhunt3r
V2EX  ›  程序员

merge 遇到 conflict 应该由哪边 resolve ?

  •  
  •   b00tyhunt3r · Sep 3, 2021 · 2297 views
    This topic created in 1713 days ago, the information mentioned may be changed or developed.

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

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

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

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

    但是作为提交方有时候摸不准冲突的部分能不能动
    merge 方直接改的话不需要扯皮效率会高很多
    yuguorui96
        10
    yuguorui96  
       Sep 4, 2021
    @b00tyhunt3r 那你修改代码前要明确改动,与 merge 方同步再做他们允许的逻辑修改,而不是先自己闷头改。
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   3843 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 39ms · UTC 04:28 · PVG 12:28 · LAX 21:28 · JFK 00:28
    ♥ Do have faith in what you're doing.