修改PR并关闭原始文件

时间:2018-12-09 10:28:50

标签: git github merge commit

我已经按照this description签出了GitHub PR的本地分支。建议

git fetch origin pull/ID/head:BRANCHNAME

在该本地分支上,我已应用了一些修复程序,并且想

  • 合并所有提交的内容(原始PR和本地PR)
  • 关闭原始PR,以便可以看到它已受到照顾(并受到赞赏)

将采取哪些适当步骤?

我正在考虑将本地分支推送到新的远程分支,创建新的PR,并将其与“ closes#original PR”消息合并。这样行吗?它会有什么不良影响?我可以使用合并消息关闭原始PR,还是必须提交?

1 个答案:

答案 0 :(得分:2)

我想说的是,如果您对此PR所做的更改更像是修饰性的\小修正,那么您应该:

  • 检出PR的源分支(克隆PR作者创建的 fork git clone https://github.com/USERNAME/REPO && cd REPO && git checkout BRANCHNAME
  • 进行更改
  • 直接将其推送到fork repo
  • 更改将显示在原始PR中
  • 合并原始PR

因此您可能不需要为此创建新的PR,只需修改现有的PR。

有关参考https://help.github.com/articles/committing-changes-to-a-pull-request-branch-created-from-a-fork/,请参见Github页面。尽管要能够提交到fork分支,您需要PR创建者选中“允许维护者进行编辑”复选框。

但是在其他情况下(例如,如果原始PR有点混乱或正在等待很长时间,并且需要立即重新定标;或者作者没有提出“允许维护者进行编辑”),那么您建议的方法就很好同样,创建新的PR并合并它而不是原来的PR也没有错。在这种情况下,我建议您仅对原始PR进行评论,例如“转移到新PR#123,关闭此PR”,这样就很清楚您在做什么。