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

正式员工挂着实习生的 MR,等自己晚开发的分支合入了再让其解冲突完合入,这是正常现象吗

  •  
  •   finnlewis · 2021-06-29 23:28:02 +08:00 · 3604 次点击
    这是一个创建于 1003 天前的主题,其中的信息可能已经有所发展或是发生改变。

    本人实习生,在某大厂实习,已经经历过 3 个不同正式员工要我解冲突,而不是自己解这种事了

    具体 timeline 是,不考虑休息日,假设我在第 1 天开始开发需求 1,第 3 天正式员工 A 开始开发需求 2,第 4 天我完成需求 1 提 MR,正式员工 A review 我的 MR 并提出若干建议,当天我完成修改并更新 MR (更新 MR 后 A 会收到消息提醒)。此后我的 MR 状态一直无变化、无新 CR 评论。第 6 天正式员工 A 完成需求 2 并提 MR,当天合入。此时我的 MR 已显示有冲突需要解决。可以认为,这三天我的分支都是在等着正式员工 A 代码合入后再被做出是否合入的决定的。冲突并不会多,我只是好奇为什么在这种情况下,正式员工不偏向于按照正常代码合入顺序来决定该谁解决冲突,而是偏向于让他人代劳。

    20 条回复    2021-07-01 01:48:34 +08:00
    shiji
        1
    shiji  
       2021-06-29 23:38:52 +08:00
    要不你下次改完了 ping 一下 A 。
    我们公司 jira bitbucket 系列的邮件巨多,大部分人要么忽略,要么单独弄个收件夹规则,没到实在没事儿干的情况不会去看里面的邮件。
    jones2000
        2
    jones2000  
       2021-06-29 23:39:23 +08:00
    只能说你们太有钱了, 小公司基本 1 个模块就对应一个人, 每个模块都是单独的文件, 根本就不会涉及到冲突. 除非是项目的工程文件有可能需要合并, 但这些都是很少的. 也不涉及到代码, 都是配置性的东西.
    liprais
        3
    liprais  
       2021-06-29 23:49:41 +08:00 via iPhone
    解冲突又不难,就解呗
    公司不是学校,同事也不是老师
    BrettD
        4
    BrettD  
       2021-06-29 23:52:32 +08:00
    因为你好欺负 :)
    finnlewis
        5
    finnlewis  
    OP
       2021-06-29 23:59:40 +08:00
    @BrettD 看来以后得嗯气起来了
    foam
        6
    foam  
       2021-06-30 00:06:14 +08:00 via Android
    同 1 楼。下次改完了要往下走的时候,记得 push 下对应的同学,因为别人可能是太忙,忘了。
    个人认为,贪图方便让实习生代劳的可能性不大。毕竟新人不熟悉业务,解决冲突的时候,万一把自己刚写的代码给干掉了,就更麻烦了。
    SingeeKing
        7
    SingeeKing  
       2021-06-30 00:08:54 +08:00 via iPhone
    别多想真的就是单纯没看到… 一天各种乱七八糟通知几十甚至有时候上百封邮件根本看不过来,我都是直接自动归档来自内部 git 通知邮件的
    dayeye2006199
        8
    dayeye2006199  
       2021-06-30 01:55:55 +08:00   ❤️ 1
    只要我的手速够快,冲突就找不上我。

    下次记得 ping 一下 reviewer 。还有如果不是功能或者架构性质的意见,(例如只是改改格式,重命名一下函数名之类的没营养的 comment ),建议直接 approve 之后,交给 MR 作者改正更新后直接自行合入。
    allan888
        9
    allan888  
       2021-06-30 02:04:59 +08:00
    "可以认为,这三天我的分支都是在等着正式员工 A 代码合入后再被做出是否合入的决定的。"
    这个应该不是,一般就只是单纯不知道你需要重新 review 而已,像楼上说的,下次改完了记得 ping 一下 A
    AntiGameZ
        10
    AntiGameZ  
       2021-06-30 02:05:51 +08:00
    冲突一般都是谁合并的时候发现谁解决吧。你这个事情的特殊之处,是你认为 A 明知道你的 MR 存在并且冲突来自于他的代码。为什么他不和你协调好 merge 顺序,是吗?
    ryd994
        11
    ryd994  
       2021-06-30 08:37:11 +08:00
    1. 正常来说不应该有大量冲突,否则你们架构可能没搞好
    2. A 可能就是忙着写码没考虑你的事情
    3. 这种情况下你应该找别人 review,而不应该傻等
    glfpes
        12
    glfpes  
       2021-06-30 09:45:35 +08:00   ❤️ 3
    很简单,因为他先 merge 了。

    只有一个准则:谁的 MR 与主干冲突,谁自己在 MR 里处理。

    至于为啥你的 MR 提交的早却 merge 的晚,那是另外 1 个问题。
    sam90
        13
    sam90  
       2021-06-30 10:05:19 +08:00   ❤️ 2
    真是戏多, 能沟通解决的事情, 你非要别人来将就你?
    hbin
        14
    hbin  
       2021-06-30 12:12:17 +08:00
    12 楼说的对,你需要解决的是另一个问题
    mara1
        15
    mara1  
       2021-06-30 13:44:33 +08:00
    @finnlewis 你说的 MR, 是不是“Pull request”? 我一直简称“PR", 这就是谁的 PR 有冲突谁解决啊, 你 PR review 完了就赶紧 merge 进去。
    zengxs
        16
    zengxs  
       2021-06-30 14:02:13 +08:00
    @mara1 这个功能在 GitHub,BitBucket 里面叫 Pull request,GitLab 里面叫 Merge Request

    一直感觉 Merge request 这个叫法更易懂点。
    刚用 GitHub 的时候看着这个 pull request 摸不着头脑,也不知道到底是 request 个啥 😂
    mara1
        17
    mara1  
       2021-06-30 14:41:34 +08:00
    @zengxs ,感谢解惑, 涨见识了,我没有用过 GitLab 哈哈。
    dingyaguang117
        18
    dingyaguang117  
       2021-06-30 22:50:51 +08:00
    赞同 12 楼。 你应该催着把自己的代码先合并了,而不是等着。。
    HankAviator
        19
    HankAviator  
       2021-07-01 01:08:01 +08:00 via Android
    冲突的确是谁遇到谁处理,但感觉已有的 mr 最好能占着坑避免同时开发
    MintZX
        20
    MintZX  
       2021-07-01 01:48:34 +08:00   ❤️ 1
    很正常啊,一般分给实习生的任务可能本来就不是很重要。基本上就是能有最好没有也暂时无所谓的。这个时候大家肯定偏向先解决主要问题。

    另外一个就是信任问题,实习生一般有一个 mentor 。其他人对实习生没有任何了解,所以一般不倾向给你 approve 。如果你有 mentor 的话,那么一般是你的 mentor 给你 approve,同时他也要承担你代码炸了服务器的后果。

    说白了,不是人家故意让你去处理分支冲突,而是一般正式员工根本就不想给实习生 approve 任何代码让他做提交。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   3469 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 29ms · UTC 00:46 · PVG 08:46 · LAX 17:46 · JFK 20:46
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.