blog icon indicating copy to clipboard operation
blog copied to clipboard

github上如何修改和合并改别人的pull request

Open aermin opened this issue 5 years ago • 4 comments

在本地修改pull request

场景:同学A提交了一个pr,同学B想把这个pr拉取到本地,并在这基础上进行修改,最后提交新的pr

  1. 在repo中点击这个

image

  1. 然后选择你要修改和合并的pr

image

  1. 滑到底部点击链接:command line instructions.

image

  1. 然后可看到github给出的接下去的操作

image

其实就是创建一个分支,然后pull拉取同学A远程repo中提交该pr的的分支,这时本地仓库代码就是同学A提交的那个pr时的版本的仓库代码。 接着你可以进行修改,然后按照你的开发流程最终把你的修改也提交一个pr。

image

示例

git checkout -b caffffe-fix-it 2.0
git pull https://github.com/caffffe/zendesk-cti-widget.git fix-it

一顿修改......

 git add .
git commit -m "increase timeout"
git push --set-upstream origin caffffe-fix-it

参考

Checking out pull requests locally

aermin avatar Aug 05 '18 14:08 aermin

你好, 我想提问, 如果有人对我的PR修改了, 然后我应用了他的修改并且commit了, 那最后的push动作由谁来执行呢?

zhuoqinyue avatar Jan 11 '23 06:01 zhuoqinyue

你好, 我想提问, 如果有人对我的PR修改了, 然后我应用了他的修改并且commit了, 那最后的push动作由谁来执行呢?

由他把他所修改的commit push到你的分支

aermin avatar Jan 11 '23 11:01 aermin

感谢, 但是最后发生的情况并不是这样的. 我复述一下吧.

我提了个PR, 然后审核者给出了修改代码, 我直接应用, 并且在PR的界面里直接commit. 后面发现, 这段代码是合并到我的这次commit里面, 只要审核者merge, 就有最新的修改代码了.

zhuoqinyue avatar Jan 13 '23 07:01 zhuoqinyue

感谢, 但是最后发生的情况并不是这样的. 我复述一下吧.

我提了个PR, 然后审核者给出了修改代码, 我直接应用, 并且在PR的界面里直接commit. 后面发现, 这段代码是合并到我的这次commit里面, 只要审核者merge, 就有最新的修改代码了.

没听懂你说的😂修改的commit是他的吧,然后push到你的pr,然后你的pr最后应该会被squash merge

aermin avatar Jan 17 '23 01:01 aermin