公開しているツールに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でした。