Skip to content

Instantly share code, notes, and snippets.

@koichik
Created Sep 29, 2011
Embed
What would you like to do?
プルリクエストを処理するときのチェックリスト

プルリクエストを処理するときのチェックリスト

まずは内容を確認して、問題なさげなら試してみよう。

  • パッチは master 用か v0.4 用か? (master にリクエストされても v0.4 に適用できるならそうする)
  • パッチを確認するためのブランチを作る。
  • パッチを適用する。
  • ビルドしてテスト。
  • コミットログに Fixes #1234 と追記するなど。

push する前に確認

  • CLA にサインしているか?
  • master まはた v0.4 の最新を pull
  • パッチを適用したブランチを rebase
  • ビルドしてテスト。
  • master または v0.4 に merge
  • ビルドしてテスト。
  • git show
  • git log
  • git status
  • 本当に大丈夫?
  • CLA をもう一度確認してみよう。
  • なんてやってる間にリモートが更新されたりしてないかな?
  • 見落としはないよね?
  • よっしゃ、git push だ!
  • GitHub に「Thanks」ってコメントするなど。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment