原来在 GitHub 上创建 pr 之前是需要先 fork 的

2021-08-17 18:15:18 +08:00
 shanghai1943

如题。之前没有在 GitHub 上给别人的项目提交过 pull request,然后把别人项目 clone 到本地拉分支修改后,想创建一个 pr,然而怎么都没法创建,最后是试着先 fork 了一下,然后把 fork 后的项目修改提交后,到原始项目里创建 pr 就可以看见 diff 了,然后就可以创建 pr 成功了。。

2367 次点击
所在节点    git
15 条回复
Senorsen
2021-08-17 21:35:11 +08:00
不 fork 你的代码能 push 到哪儿去,你对别人的项目仓库又没有写权限的…
jdhao
2021-08-17 21:36:14 +08:00
除非你是这个项目的 maintainer
yeqizhang
2021-08-17 21:52:43 +08:00
比较好奇你第一次是怎么创建 pr 的……
Veneris
2021-08-17 22:07:47 +08:00
我印象里如果是单文件小改动,也就是直接用 github 的编辑的话,就会自动 fork 然后 pr 。

后来码云还弄了个快速 pr(忘记叫什么了,大概类似的名字),好像就是不 fork,直接 pr,似乎 gthub 现在也支持?
GuuJiang
2021-08-17 23:06:04 +08:00
pr 既然叫做 pull request,表示的是请求 maintainer 去 pull,如果你不 fork 的话准备让人从哪去 pull 呢?
john6lq
2021-08-18 02:55:54 +08:00
我即使是所谓的 maintainer,但感觉为了协作流程更规范还是应该 fork 再提 PR 。当然自己一个人怎么都行。
LotusChuan
2021-08-18 03:00:48 +08:00
可以看一下 Pro Git,GitHub 的人写的。
dcoder
2021-08-18 03:17:38 +08:00
说实话, 我当年一次搞的时候, 也是懵逼, 没啥不好意思问的
kuangwinnie
2021-08-18 03:29:51 +08:00
@john6lq 对,项目本身应该是个类似法人那样的存在,maintiner 只是救火的。
msg7086
2021-08-18 03:30:52 +08:00
@john6lq 单 repo 没必要搞 PR,用 MR 就行了,这和流程规范没关系。
IgniteWhite
2021-08-18 03:32:50 +08:00
应该发在 新手求助 节点
shanghai1943
2021-08-18 09:49:18 +08:00
@yeqizhang #3 刚开始我是想在原 repo 里创建 pr,虽然可以看到创建按钮,但是没法实际创建,因为没有其他的分支可以合并进原 repo 里。
shanghai1943
2021-08-18 09:50:33 +08:00
@Senorsen #1 说的在理。之前本地项目都是基于 master 来拉分支,然后修改提交合并。把 GitHub 也想成这种模式了。
shanghai1943
2021-08-18 09:50:46 +08:00
@Veneris #4 这个倒是没有试验过。
shanghai1943
2021-08-18 09:51:07 +08:00
@dcoder #8 😂 感谢鼓励

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

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

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

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

© 2021 V2EX