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

2021-06-29 23:28:02 +08:00
 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 代码合入后再被做出是否合入的决定的。冲突并不会多,我只是好奇为什么在这种情况下,正式员工不偏向于按照正常代码合入顺序来决定该谁解决冲突,而是偏向于让他人代劳。

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

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

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

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

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

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

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

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

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

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

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

© 2021 V2EX