share facebook facebook2 twitter menu hatena pocket slack

2016.08.24 WED

Github.comに初めてPull Requestがきた

古渡 晋也

WRITTEN BY 古渡 晋也

公開しているツールにPull Requestがきました。

というか、社員の@inokappaさんからきました。
こちらが プルリク
リポジトリは、こちら。 AWS PDFをダウンロードするツールのリポジトリ

マージする

Pull Requestのページに手順があった。便利。

マスターを取得する

[f_prg@work_repo] $ https://github.com/inokappa/aws-doc-pdf-download.git
Cloning into 'aws-doc-pdf-download'...
remote: Counting objects: 81, done.
remote: Total 81 (delta 0), reused 0 (delta 0), pack-reused 81
Unpacking objects: 100% (81/81), done.
Checking connectivity... done.
[f_prg@work_repo] $ cd awe-doc-pdf-download

チェックアウト

Step 1: From your project repository, check out a new branch and test the changes.

[f_prg@aws-doc-pdf-download] $ git checkout -b inokappa-yaml_url_list master
Switched to a new branch 'inokappa-yaml_url_list'
[f_prg@aws-doc-pdf-download] $ git pull https://github.com/inokappa/aws-doc-pdf-download.git yaml_url_list
From https://github.com/inokappa/aws-doc-pdf-download
 * branch            yaml_url_list -> FETCH_HEAD
Updating 6d8c985..65e0f16
Fast-forward
 Dockerfile  |   7 ++
 README.md   |  29 ++++-
 configs.yml | 190 +++++++++++++++++++++++++++++++++
 scraping.rb | 349 +++---------------------------------------------------------
 4 files changed, 240 insertions(+), 335 deletions(-)
 create mode 100644 Dockerfile
 create mode 100644 configs.yml

検証

dockerコマンドが全然わからなかった。
@inokappaさんからほとんど教えていただいた。
勉強しないと。

マージ

Step 2: Merge the changes and update on GitHub.
なんか、うまくできなかった。また次の機会に。

GitHub.comでマージする

Merge Pull Requestをポチっと。

まとめ

以上、初めてのPull Requestでした。

元記事はこちら

Github.comに初めてPull Requestがきた

古渡 晋也

古渡 晋也

縁があって、cloudpackにジョインしました。 JAWS-UG さいたま支部コアメンバーとしても活動しております。 cloudpackにてインフラ設計・構築・運用をしております。 HadoopやMongoDBなどいろいろ経験しております。 あと、開発もできるエンジニアです。 開発可能な言語はC, C++, C#, アセンブラ, PHP, Ruby, JavaScript, Java, Objective-Cです。