-
Notifications
You must be signed in to change notification settings - Fork 547
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pull Request 与 Merge Request 的区别 #37
Comments
我也觉得名字起的不好,我第一次去提 pr 时候,差点跑到对方项目里面去提一个 |
名字花里胡哨,背后的道理往往很简单。大家都喜欢把名字起的很高大上,这样显得“专业” |
我就觉得你最开始说的很有道理 后面有点扯,但这才是真相😂 |
我也觉得MR表达的更清晰,PR给我反向操作的感觉 |
|
真的 |
没人觉得fork也莫名其妙吗 |
fork还好吧 类Unix系统里那么多 |
github的pull request ,github上的项目都是开源库,要想参与,就必须先fork别人的项目,然后提pr。如果你是项目的参与者,可以使用同一个仓库的分支进行pr,这两个词,确实没什么区别。 |
✅[Pull Request 与 Merge Request 的区别](brickspert/blog#37)
我理解其实是主体问题吧,pull request 的主语是代码owner,而不是贡献者 :) |
清晰简洁,明白了,多谢 |
这篇文章只为说明一个问题:“Pull Request 与 Merge Request 有什么区别?”
在我的想象中,有一双滑板鞋~不好意思,跑偏了。在我的想象中,它俩肯定是不一样的,并且大部分人的想法应该和我是一样的,我先来说说我的想法。
自我 YY
如果经常用 Github,一定十分了解 Pull Request。
如果经常用 Gitlab,一定十分了解 Merge Request。
基于对 Github 和 Gitlab 的了解,我潜意识里感知到 Pull Request 与 Merge Request 是有区别的。
Github 一般是公开库,当然没有人愿意别人直接在自己的仓库上面修改代码。所以我们如果要给别人的仓库贡献代码,一般是要 fork 一个仓库,在自己的仓库改完后,给原仓库提交 PR 请求,请求原仓库主人把你的代码拉(pull)回去。
下图是一般的 Github 工作流程。
Gitlab 一般是私有库,一个团队维护一个仓库,通常大家会新建自己的分支,开发完成后,请求合并回主干分支。
下图是一般的 Gitlab 工作流程。
基于上面的认知,我起初觉得
“我改了你们的代码,你们拉回去看看吧 !”
“请求合并代码!”
说实话,我自己都说服不了自己,上面的理解是正确的。毕竟
真实情况
我们来看看 gitlab 官方是怎么说的:
大概意思就是 Merge Request 和 Pull Request 是同一个东西,仅仅只是名字不一样。
一般我们执行分支合并,需要执行下面两个命令:
Github 选择了第一个命令来命名,叫 Pull Request。
Gitlab 选择了最后一个命令来命名,叫 Merge Request。
这个理由是 Gitlab 官方给的,我觉得还是可信的。所以我们的结论就是**“Pull Request 和 Merge Request”是一个东西**。
吐槽
Pull Request 这个词起的真不好!我想起来我刚开始用 Github 的时候,根本看不懂 Pull Request 是干啥的。
如果我来起名的话,我应该会起这几个名字:
无论如何也想不到 Pull Request,一个好名字还是非常非常重要的。网上能搜到很多人问 Pull Request 是什么意思,然后大家的解释都差不多“请求别人拉你的代码”。如果当时不用这个名字,大家一看就明白了,也不用问了。
又让我想起了小白时期被 redux 中的名词支配的恐惧,
reducer
等新名字,看的我一愣一愣的。就像我在 github 的简介中写的话一样,一切都是纸老虎。
你有没有被奇奇怪怪的名字支配的恐惧呢?
如果是你,你会给 Pull Request 取什么名字呢?
欢迎留言互动,让大家看到你的想法。
❤️感谢大家
关注公众号「前端技术砖家」,拉你进交流群,大家一起共同交流和进步。
The text was updated successfully, but these errors were encountered: