GitHubでPullRequestを受けた時にどうするか

AXRatingViewやAXStretchableHeaderTabViewControllerといったOSSを公開してから、GitHub上でPullRequestを受けることが多くなった。細かい修正が多かったので動作確認せずマージしていたんだけど、動作確認が必要なPullRequestが増えてきたので、PullRequestを受けた時にどうするべきか調べた。ちなみに、これは自分以外のリポジトリからPullRequestを受け入れる場合の話で、例えば同じリポジトリの別ブランチに向けたPullRequestの話ではない。

  • PullRequestの動作確認
  • コードレビュー
  • Merge pull requestして感謝

参考文献:
http://akisute.com/2011/02/github-pull-request.html

PullRequestの動作確認

以下の操作で、新しく作ったブランチに別リポジトリの指定したブランチの内容を取り込める。

$ git checkout develop
$ git checkout -b pr/bug-fix
$ git pull https://github.com/example/Example.git issues/bug-fix

コードレビュー

  • お好きなように。

Merge pull request

  • イエーイ

Merge pull request


「GitHubでPullRequestを受けた時にどうするか」への2件のフィードバック

  1. プルリクの一番下のところに、「手元でCLIでmergeする方法はこちら」ってリンクがあって押すとcheckoutしてpullして〜〜ってコマンドがでてくるので俺それコピペしてやってる

  2. たしかにリンクありますね…。そのやり方の方が正しいっぽいです。

コメントを残す