feat(ch3): Pull Requestについて追記

Author: kubosho

src/chapters/chapter3.md

@@ -44,20 +44,27 @@ Issue は現在のリポジトリにある問題点や改善点をみんなで
 
 ## Pull Request
 
-Steins;Gate で桐生萌郁が D メールを送る直前に文言を差し替えたことは、変更が問題ないと承認されてから変更を加えたことになります。
-承認されていない変更なのでバグが起きる可能性もあります。実際に Steins;Gate では世界線変動が起きて岡部倫太郎の主観からは桐生萌郁が消えたように見えました。なので、承認されたあとは何も変更を加えないようにするのが良いと言えます[^3]。
+Pull Request は他のリポジトリに対して、問題点や改善したい点の修正をしたときに、その修正を取り込んでもらうよう依頼するときに使います。
+例として自分が vivliostyle-ui というリポジトリに対して送った Pull Request[^4] を見てみます。
+これはもともと JavaScript で書かれたコードを TypeScript に変換する Pull Request です。
 
-### コードレビュー
+![vivliostyle-uiに対して送ったJavaScriptで書かれたコードをTypeScriptへ変換するPull Request](https://i.gyazo.com/25ef15b95a8e7055b4e3c142461c87e6.png)
 
-コードレビューの機能は岡部倫太郎以外が D メールや D ラインを送る場合に岡部倫太郎へ文言を見せる行為のようなものです。
-現在のリポジトリにある問題点や改善点の修正を取り込んでもらうように依頼するのに使います。
+このように Pull Request の概要欄にこれはどのような Pull Request で、どういったきっかけでやることにしたのか、またやったことの詳細などが書かれています。
 
+つまり Pull Request はラボメンが D メールや D ラインを送る場合に岡部倫太郎へ文言を見せたあと D メールや D ラインを送る行為です。
 たとえば桐生萌郁が「携帯の機種変に失敗したので前の携帯を使い続ける」旨の D メールを送ろうとしたとき、岡部倫太郎に見せてから D メールを送りました。
-これはチームでの開発において、作業内容をコミットしてプッシュした後に、他のメンバーへ変更点を伝えて何かあれば修正してからマージする流れと同じです。
+これと同じことを GitHub 上でも Pull Request という形でおこなえます。
+
+コードを見る人は見た結果を Pull Request を送った人に対して伝えられます。
+これをコードレビューと言います。自分のリポジトリでない限りは送った Pull Request に対してコードレビューがあります。
 
-Code review にはその変更が問題ないと承認する機能があります。またコードを書いた人にコードを直してほしい場合に変更をリクエストする機能もあります。
+![ただコメントするだけか、Pull Requestを承認または一旦拒否するかを選べる](https://i.gyazo.com/4c8c0a97a163d4f821ef425c6fe24945.png)
 
-### ブランチの保護
+Steins;Gate で桐生萌郁が D メールを送る直前に文言を差し替えたことは、変更が問題ないと承認されてから変更を加えたことになります。
+承認されていない変更なのでバグが起きる可能性もあります。実際に Steins;Gate では世界線変動が起きて岡部倫太郎の主観からは桐生萌郁が消えたように見えました。
+そのため承認された後は何も変更を加えないようにするのが良いと言えます。
+実際に GitHub 上にはコードレビューをした後にあらためてコードレビューをおこなわないとマージをおこなえなくする設定があります。
 
 ## Project
 
@@ -71,6 +78,7 @@ Project はオペレーションです。Project を使うことにより Issue
 [^1] https://www.kikakurui.com/q/Q27000-2019-01.html
 [^2] https://capec.mitre.org/data/definitions/253.html
 [^3] https://github.com/o2project/steins-git/issues/18
+[^4] https://github.com/vivliostyle/vivliostyle-ui/pull/79
 [^4] GitHub 上の設定として、変更が問題ないと承認された後にコードを変更した場合に再度承認をもらわないとマージできないようにする設定があります。
 
 <!-- textlint-disable -->