git
Pro Git
Atlassian Git Tutorial
Pro Git 简体中文翻译
GitX
shanghai1943
V2EX  ›  git

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

  •  
  •   shanghai1943 · Aug 17, 2021 · 2935 views
    This topic created in 1754 days ago, the information mentioned may be changed or developed.

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

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

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