接收Pull Request--第7章学习笔记

7.1 采纳Pull Request的方法

点击Merge pull request按钮,Pull Request的内容便会自动合并至仓库。在采纳之前,请尽量将接收到的Pull Request拿到本地开发环境中进行检查,确认是否能够正常运行以及代码是否安全。或者用将要在第8章中介绍的Jenkins等持续集成工具进行自动测试,保证新代码不破坏原有功能之后,再合并进仓库。

7.2 采纳Pull Request前的准备

1、确认Pull Request送来的代码是否运行正常。

2、代码审查

在GitHub上可以对Pull Request的具体的某行代码进行评论。

3、在本地开发环境中反映Pull Request的内容

将接收方的本地仓库更新至最新状态
clone及fetch

首先,将PR接收方的仓库clone到本地,如果已经clone过,那么请进行pull 等操作更新至最新状态。

获取发送方的远程仓库

将PR发送方的仓库设置为本地仓库的远程仓库,获取发送方仓库的数据。在本例中,我们将图7.8右上的仓库设置为远程仓库,进行fetch。

$ git remote add PR发送者 git@github.com:PR发送者/first-pr.git
$ git fetch PR发送者
创建用于检查的分支

我们需要创建一个分支,用来模拟采纳PR后的状态。分支名暂定为pr1,现在gh-pages与pr1分支的内容完全相同。

checkout
$ git checkout -b pr1
合并

将已经fetch完毕的“PR发送者/work”的修改内容与pr1分支进行合并。也就是图7.9下侧箭头(merge)代表的操作。

$ git merge PR发送者/work

这样一来,pr1分支中就加入了“PR发送者/work”分支的修改内容,在实际开发中,各位需要通过自动测试等手段检查软件是否能正常运行。即运行、测试pr1分支的代码。

删除分支

检查结束后pr1分支就没用了,可以直接删除。我们切换至pr1之外的分支,运行下面的代码。

$ git branch -D pr1

7.3 采纳Pull Request

采纳PR有两种方式:自动合并,在GitHub中直接点击Merge pull request按钮;手动合并,在本地通过CLI进行合并操作再push至GitHub。

下面是手动合并的方法:

合并到主分支

步骤:

1、切换至主分支(本例中为gh-pages)

2、合并“PR发送者/work”分支的内容

$ git checkout gh-pages
$ git merge PR发送者/work
push修改内容

1、在push之前,为保险起见,我们先查看本地与GitHub端仓库代码的差别。

2、确认没有目的之外的差别后,进行push。

$ git diff origin/gh-pages
$ git push

用这种方法处理后,仓库的Pull Request会自动从Open状态变为Close状态。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,684评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 87,143评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 151,214评论 0 337
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,788评论 1 277
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,796评论 5 368
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,665评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,027评论 3 399
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,679评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 41,346评论 1 299
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,664评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,766评论 1 331
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,412评论 4 321
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,015评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,974评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,203评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,073评论 2 350
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,501评论 2 343

推荐阅读更多精彩内容