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

[finnlewis]

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

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

[shiji]

要不你下次改完了 ping 一下 A 。
我们公司 jira bitbucket 系列的邮件巨多，大部分人要么忽略，要么单独弄个收件夹规则，没到实在没事儿干的情况不会去看里面的邮件。

[jones2000]

只能说你们太有钱了, 小公司基本 1 个模块就对应一个人, 每个模块都是单独的文件, 根本就不会涉及到冲突. 除非是项目的工程文件有可能需要合并, 但这些都是很少的. 也不涉及到代码, 都是配置性的东西.

[liprais]

解冲突又不难，就解呗
公司不是学校，同事也不是老师

[BrettD]

因为你好欺负 :)

[finnlewis]

@BrettD 看来以后得嗯气起来了

[foam]

同 1 楼。下次改完了要往下走的时候，记得 push 下对应的同学，因为别人可能是太忙，忘了。
个人认为，贪图方便让实习生代劳的可能性不大。毕竟新人不熟悉业务，解决冲突的时候，万一把自己刚写的代码给干掉了，就更麻烦了。

[SingeeKing]

别多想真的就是单纯没看到… 一天各种乱七八糟通知几十甚至有时候上百封邮件根本看不过来，我都是直接自动归档来自内部 git 通知邮件的

[dayeye2006199]

只要我的手速够快，冲突就找不上我。

下次记得 ping 一下 reviewer 。还有如果不是功能或者架构性质的意见，（例如只是改改格式，重命名一下函数名之类的没营养的 comment ），建议直接 approve 之后，交给 MR 作者改正更新后直接自行合入。

[allan888]

"可以认为，这三天我的分支都是在等着正式员工 A 代码合入后再被做出是否合入的决定的。"
这个应该不是，一般就只是单纯不知道你需要重新 review 而已，像楼上说的，下次改完了记得 ping 一下 A

[AntiGameZ]

冲突一般都是谁合并的时候发现谁解决吧。你这个事情的特殊之处，是你认为 A 明知道你的 MR 存在并且冲突来自于他的代码。为什么他不和你协调好 merge 顺序，是吗？

[ryd994]

1. 正常来说不应该有大量冲突，否则你们架构可能没搞好
2. A 可能就是忙着写码没考虑你的事情
3. 这种情况下你应该找别人 review，而不应该傻等

[glfpes]

很简单，因为他先 merge 了。

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

至于为啥你的 MR 提交的早却 merge 的晚，那是另外 1 个问题。

[sam90]

真是戏多, 能沟通解决的事情, 你非要别人来将就你?

[hbin]

12 楼说的对，你需要解决的是另一个问题

[mara1]

@finnlewis 你说的 MR, 是不是“Pull request”？ 我一直简称“PR", 这就是谁的 PR 有冲突谁解决啊， 你 PR review 完了就赶紧 merge 进去。

[zengxs]

@mara1 这个功能在 GitHub，BitBucket 里面叫 Pull request，GitLab 里面叫 Merge Request

一直感觉 Merge request 这个叫法更易懂点。
刚用 GitHub 的时候看着这个 pull request 摸不着头脑，也不知道到底是 request 个啥 😂

[mara1]

@zengxs ,感谢解惑， 涨见识了，我没有用过 GitLab 哈哈。

[dingyaguang117]

赞同 12 楼。 你应该催着把自己的代码先合并了，而不是等着。。

[HankAviator]

冲突的确是谁遇到谁处理，但感觉已有的 mr 最好能占着坑避免同时开发

[MintZX]

很正常啊，一般分给实习生的任务可能本来就不是很重要。基本上就是能有最好没有也暂时无所谓的。这个时候大家肯定偏向先解决主要问题。

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

说白了，不是人家故意让你去处理分支冲突，而是一般正式员工根本就不想给实习生 approve 任何代码让他做提交。