Github Pull Request的提出与采纳

这一文来简要介绍一下Github Pull Request（如下简称PR）的使用方法：python

做为PR的提出者，如何对某个仓库提交PR，如何根据仓库管理者对所提交PR的反馈对PR进行完善
做为PR的接收者，如何对PR进行测试，对提出者进行反馈以及合并PR到仓库中。

这里我使用两个GitHub帐户来进行说明，PR接收者帐户为lml256，PR提出者帐户为rikkaii。并以lml256帐户中的learngit仓库进行试验。git

如何提出PR

若是你对Github上的某个开源项目很是感兴趣，想贡献本身的力量为其添加新功能，或者发现了代码中的一些bug，想为其进行修复。那么能够将该开源仓库Fork到你的仓库列表中，并对其进行修改后，向原仓库提交PR，请求仓库的管理员合并你的代码。下面假设咱们的帐号为rikkaii，并向lml256帐号中的learngit项目提交PR为例，详细的说明一下该流程：bash

Fork仓库

Fork仓库很是简单，只须要到仓库首页的右上角点一下Fork按钮便可，Github会自动在你的仓库列表中建立该仓库的一个副本。测试

以下图，GitHub自动在你的帐户上建立了一个副本，并在仓库名的下方指明了该仓库是Fork来的：fetch

添加新功能

如今咱们为该仓库添加一些新功能：spa

首先将该仓库clone到本地3d

$ git clone git@rikkaii:rikkaii/learngit.git
Cloning into 'learngit'...
remote: Enumerating objects: 65, done.
remote: Total 65 (delta 0), reused 0 (delta 0), pack-reused 65
Receiving objects: 100% (65/65), 5.44 KiB | 928.00 KiB/s, done.
Resolving deltas: 100% (17/17), done.

咱们从master分支上新建一个特性分支，并在特性分支上面实现咱们的新功能。这里简要说明下什么是特性分支：在主流的git开发模式中并不会直接在master上开发，而是在开发某个功能时新建一个分支，叫作特性分支，在其上进行开发，等到开发完成并测试完成后，再合并到master分支中，而后删除该分支。而且特性分支的名称要简明扼要，方便别人明白其做用。这里咱们建立一个python-div分支用于开发新功能。code

$ git checkout -b python-div
Switched to a new branch 'python-div'

咱们在目录中建立一个div.py文件并在其编写一些代码：blog

# div.py

# div.py

def div(a, b):
    ans = a / b
    return ans

而后进行提交，并推送到GitHub：开发

$ git add div.py

$ git commit -m "add div.py"
[python-div 7aad1de] add div.py
 1 file changed, 5 insertions(+)
 create mode 100644 div.py

$ git push --set-upstream origin python-div
...
To rikkaii:rikkaii/learngit.git
 * [new branch]      python-div -> python-div
Branch 'python-div' set up to track remote branch 'python-div' from 'origin'.

在上面的命令中，因为python-div分支是咱们本地新建的，在GitHub远程仓库中并无该分支的相关信息，为了可以将该分支的修改推送到GitHub远程仓库上，在推送时须要添加--set-upstream参数，这样会在远程仓库中也建立一个python-div分支，并将该分支的修改推送上去。

能够看到，咱们的仓库中已经有了python-div分支以及本次commit添加的div.py文件。

发起PR

接下来，你能够点击上图中分支名称旁边的的New pull Request按钮来发起一个PR。这会自动跳转到原仓库下面，并选择python-div做为要推送的分支，以下：

至于要将python-div分支上的修改推送到原仓库的什么分支上，Github在这里自动为咱们选择了master分支，这里须要注意的是分支的选择与原仓库使用的什么开发模式有关，通常正规的开源仓库都会说明PR的提交规则，包括提交到什么分支以及如何描述commit等等，这须要咱们仔细阅读相关的要求并遵循。这里为了演示方便，将python-div分支的修改请求推送到原仓库的master上，而且在上图下面的说明栏里也只是使用GitHub的默认说明。

接下来即可以点击上图下面的Create pull request按钮来建立一个分支了。

当出现上图的样子后，咱们的PR就建立成功了。咱们只须要静静等待仓库管理员的审核就好了。

修正PR

可是，有些事老是事与愿违，有时候咱们认为咱们的代码没有问题，可是却被仓库管理员反馈（或者被其余开发者指出）咱们的代码有问题，须要咱们修复。管理员经过对咱们的代码进行审计后，指出了咱们的错误，Github也会将管理员与咱们之间及其余开发者之间（这里并无其余开发者）对该次PR的讨论反映在了时间轴上，并会以邮件的形式通知咱们：

接下来，咱们按照仓库管理员的要求，对代码进行完善，修改后的代码以下：

# div.py

def div(a, b):
    try:
        ans = a / b
    except ZeroDivisionError as e:
        print('error:', e)
        return None
    return ans

修改完成后，对代码进行提交并推送到GitHub：

$ git add div.py

$ git commit -m "fix zero division error"
[python-div c3b73e7] fix zero division error
 1 file changed, 5 insertions(+), 1 deletion(-)

$ git push
...
To rikkaii:rikkaii/learngit.git
   7aad1de..c3b73e7  python-div -> python-div

当咱们推送完后，无需再次发起PR，Github会自动的将该次的提交反映到原来PR时间轴上，以下图，而且Github还会向管理员发送邮件通知本次修改：

当仓库管理者经过了本次代码，将咱们的代码合并到了master分支中后，咱们也会收到Github发来的关于Merge本次PR的邮件通知。

接收PR

说完了如何提交PR，咱们再说一说如何接收PR。做为一个开源项目的管理者，学会如何测试别人提出的PR并进行反馈或者合并是很必要的。这里假设咱们的帐号为lml256，以接收rikkaii帐号向lml256帐号中的learngit仓库提交的PR为例，详细的说明一下做为仓库的管理者，如何测试和合并PR。

审计PR

打开咱们的仓库，在Pull Requests列表中能够看到rikkaii向咱们的仓库提交了一个PR

点进去能够看到详细的信息：

咱们能够在Files changed栏里对代码进行审计，若是代码有问题的话，能够对每行代码添加批注，以帮助PR提交者进行完善，写完批注后，点击start a review即可发起一个code review。以下：

当代码所有都审计完后，点击下图中的submit review即可以提交本次的审计结果。

提交后，咱们对代码的审计结果便会反映在该PR的时间轴上，而且GitHub会向PR发起者发送回复通知邮件。

对PR进行本地测试

只在Github上凭借眼睛看并不能发现全部的问题，在对PR正式合并前，咱们还须要对PR进行本地测试，如何使用git进行操做以下图：

首先咱们须要将咱们仓库的代码clone到本地（也可能不须要clone，若是本地已经有该仓库的话），而后拉取PR发起者仓库的相关信息，这须要咱们先将PR发起者Fork的仓库添加到本地仓库的远程仓库：

$ git remote add PR_Sponsor git@rikkaii:rikkaii/learngit.git

$ git fetch PR_Sponsor  # 获取PR发起者仓库的相关信息
...
From rikkaii:rikkaii/learngit
 * [new branch]      dev        -> PR_Sponsor/dev
 * [new branch]      master     -> PR_Sponsor/master
 * [new branch]      python-div -> PR_Sponsor/python-div

能够看到，咱们从PR发起者仓库中获取到三个分支，其中包括为咱们提交PR的python-div分支。

在进行测试以前，咱们还须要建立一个特性分支用于测试PR，并将python-div分支的修改merge到特性分支上进行测试：

$ git checkout -b test_pr
Switched to a new branch 'test_pr'

$ git merge PR_Sponsor/python-div
Updating 9053d5b..c3b73e7
Fast-forward
 div.py | 9 +++++++++
 1 file changed, 9 insertions(+)
 create mode 100644 div.py

Merge代码

通过一番测试后，咱们确信该PR的代码没有问题，能够合并代码到PR发起者要求合并到的分支上（这里是master）。这里有两种方法来处理这个问题，一是直接使用GitHub提供的Merge按钮，这种方法比较直观方便，可是不够灵活，没法处理合并冲突；二是在本地将测试后的特性分支合并到要合并的分支上，而后手动提交。尽管第二种方式处理起来比较复杂，可是却比较灵活，能够处理Github不能胜任的地方。这里咱们选择第二种：

$ git checkout master
Switched to branch 'master'
Your branch is up to date with 'origin/master'.

$ git merge test_pr
Updating 9053d5b..c3b73e7
Fast-forward
 div.py | 9 +++++++++
 1 file changed, 9 insertions(+)
 create mode 100644 div.py

$ git branch -d test_pr   # 这里咱们的特性分支已没做用，能够删除
Deleted branch test_pr (was c3b73e7).

$ git push
...
To lml256:lml256/learngit.git
   9053d5b..c3b73e7  master -> master

这里，咱们已经处理完了该PR，同时，GitHub也会检测到咱们合并了该PR，也将其反映到该PR的时间轴上，并将该PR的状态设置为Merged状态，关闭该次PR：

完

版权全部，未经容许不得转载