pull-request-test

githubのプルルクエストを試す。 参考:サル先生のGit入門

リポジトリの準備

• 新しいリポジトリを作成する（pull-request-test）
• リポジトリの中に以下のjsファイル（sort.js）を作成する

var number = [19, 3, 81, 1, 24, 21];
console.log(number);

• リモートブランチにプッシュする

開発ブランチで修正

• 作業ブランチの作成

$ git check out -b add-sort-func

• ファイルを変更

var sortNumber = function (number) {
  number.sort(function (a,b) {
    if (a == b) {
      return 0;
    }
    return a < b ? -1 : 1;
  });
};

var number = [19, 3, 81, 1, 24, 21];
sortNumber(number);
console.log(number);

• コミット

$ git add sort.js
$ git commit -m "Add function list sort"

• 変更したブランチをプッシュ

$ git push origin add-sort-function

プルリクエストの作成

• githubページの対象リポジトリで「Compare & pull request」をクリック
• base:master → ターゲットブランチ（プルリクエストをマージする対象のブランチ）
• compare:add-sort-func → プルリクエストブランチ（マージしてもらうブランチ）
• Writeにリクエストの内容を入力し「Create pull request」をクリックしたら、プルリクエストが作成できる。

レビューとマージ

• レビュー担当者は「Files chaged」タブから変更内容を確認

• 修正を望む箇所でプラスボタンをクリックし、修正の依頼ができる

• 修正の依頼は「Conversation」タブでも確認することができる

• レビュー内容に従い、ソースコードを修正する（==を===に）

• 再度コミットとプッシュを行う

$ git add sort.js
$ git commit -m "(==)fix to(===)"
$ git push origin add-sort-func

• github上で、プルリクエストに修正した旨を伝える

画面上からマージ

• レビューした結果、問題なければマージを行う。「Conversation」タブより「Merge pull request」ボタンをクリック、適切なコメントを付け、「Confirm merge」でマージする

• （レビュー担当者は、開発者にコメントするとGood）

• （「Insights」タブの、「Network」より、開発ブランチが視覚的にわかる）