V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
git
Pro Git
Atlassian Git Tutorial
Pro Git 简体中文翻译
GitX
shanghai1943
V2EX  ›  git

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

  •  
  •   shanghai1943 · 2021-08-17 18:15:18 +08:00 · 2220 次点击
    这是一个创建于 976 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

    后来码云还弄了个快速 pr(忘记叫什么了,大概类似的名字),好像就是不 fork,直接 pr,似乎 gthub 现在也支持?
    GuuJiang
        5
    GuuJiang  
       2021-08-17 23:06:04 +08:00 via iPhone
    pr 既然叫做 pull request,表示的是请求 maintainer 去 pull,如果你不 fork 的话准备让人从哪去 pull 呢?
    john6lq
        6
    john6lq  
       2021-08-18 02:55:54 +08:00 via iPhone   ❤️ 1
    我即使是所谓的 maintainer,但感觉为了协作流程更规范还是应该 fork 再提 PR 。当然自己一个人怎么都行。
    LotusChuan
        7
    LotusChuan  
       2021-08-18 03:00:48 +08:00 via iPhone
    可以看一下 Pro Git,GitHub 的人写的。
    dcoder
        8
    dcoder  
       2021-08-18 03:17:38 +08:00
    说实话, 我当年一次搞的时候, 也是懵逼, 没啥不好意思问的
    kuangwinnie
        9
    kuangwinnie  
       2021-08-18 03:29:51 +08:00
    @john6lq 对,项目本身应该是个类似法人那样的存在,maintiner 只是救火的。
    msg7086
        10
    msg7086  
       2021-08-18 03:30:52 +08:00
    @john6lq 单 repo 没必要搞 PR,用 MR 就行了,这和流程规范没关系。
    IgniteWhite
        11
    IgniteWhite  
       2021-08-18 03:32:50 +08:00 via iPhone
    应该发在 新手求助 节点
    shanghai1943
        12
    shanghai1943  
    OP
       2021-08-18 09:49:18 +08:00   ❤️ 1
    @yeqizhang #3 刚开始我是想在原 repo 里创建 pr,虽然可以看到创建按钮,但是没法实际创建,因为没有其他的分支可以合并进原 repo 里。
    shanghai1943
        13
    shanghai1943  
    OP
       2021-08-18 09:50:33 +08:00
    @Senorsen #1 说的在理。之前本地项目都是基于 master 来拉分支,然后修改提交合并。把 GitHub 也想成这种模式了。
    shanghai1943
        14
    shanghai1943  
    OP
       2021-08-18 09:50:46 +08:00
    @Veneris #4 这个倒是没有试验过。
    shanghai1943
        15
    shanghai1943  
    OP
       2021-08-18 09:51:07 +08:00
    @dcoder #8 😂 感谢鼓励
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   1012 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 26ms · UTC 19:01 · PVG 03:01 · LAX 12:01 · JFK 15:01
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.