Warning: file_get_contents(/data/phpspider/zhask/data//catemap/9/git/24.json): failed to open stream: No such file or directory in /data/phpspider/zhask/libs/function.php on line 167

Warning: Invalid argument supplied for foreach() in /data/phpspider/zhask/libs/tag.function.php on line 1116

Notice: Undefined index: in /data/phpspider/zhask/libs/function.php on line 180

Warning: array_chunk() expects parameter 1 to be array, null given in /data/phpspider/zhask/libs/function.php on line 181
Git 修改请购单和结案原件_Git_Github_Merge_Commit - Fatal编程技术网

Git 修改请购单和结案原件

Git 修改请购单和结案原件,git,github,merge,commit,Git,Github,Merge,Commit,我已经根据检查了GitHub公关的一个本地分支。它表明 git fetch origin pull/ID/head:BRANCHNAME 在那个本地分支上,我已经应用了一些修复程序,希望 将所有提交合并到master(原始PR和我的本地PR) 关闭原始PR,以便可以看到它已被处理(并受到赞赏) 应采取哪些适当步骤 我正在考虑将本地分支推送到新的远程分支,创建一个新的PR,并将其与“closes#原始PR”消息合并。这样行吗?会有什么不良影响?我是否可以使用合并消息关闭原始PR,或者这必须是

我已经根据检查了GitHub公关的一个本地分支。它表明

git fetch origin pull/ID/head:BRANCHNAME
在那个本地分支上,我已经应用了一些修复程序,希望

  • 将所有提交合并到master(原始PR和我的本地PR)
  • 关闭原始PR,以便可以看到它已被处理(并受到赞赏)
应采取哪些适当步骤


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

我想说,如果您对此PR所做的更改更像是化妆品\小补丁,那么您应该:

  • 签出PR的源分支(克隆PR的作者创建的分支)
    git克隆https://github.com/USERNAME/REPO &&cd回购和git结帐分行名称
  • 做些改变
  • 直接推到fork repo
  • 更改将显示在原始PR中
  • 合并原始PR
所以,也许你不需要为此创建新的公关,只需修改现有的公关即可

请参见Github页面以获取参考。虽然为了能够提交到fork分支,您需要PR创建者选中“允许维护人员编辑”复选框


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

谢谢。但是,当我按照指示使用
git fetch origin pull/ID/head:BRANCHNAME
创建本地分支时,我现在有一个本地分支,它不是实际的PR。PR的上游分支是什么,
pull/ID/head
?根据Github,PR的创建者需要勾选“允许维护者编辑”这意味着要么我不应该创建本地分支,要么我应该签出命令行合并并在那里添加修复,对吗?现在我的选择似乎是使用合并场景重做,合并后在主控台上重做,或者创建一个新的PR。或者我甚至可以将我的本地分支合并到本地主控台上,推它,然后用漂亮的注释关闭PR,对吗?通常你有很多选择,对吧。没有真正“错误”的解决方案。这完全取决于变化的确切类型和数量。但请注意,对其他人的提交进行重定会使其签名(如果有的话)无效。如果您想保持正确签署的提交,您需要在原始PR的基础上进行更改,而不更改其他人的提交。当然,对于一些小项目来说可能不是这样,但一般来说,人们可能会关心它。