Skip to content

Instantly share code, notes, and snippets.

@koichik

koichik/gist:1250610

Created September 29, 2011 12:04
Show Gist options

  • Select an option

    Learn more about clone URLs
  • Save koichik/1250610 to your computer and use it in GitHub Desktop.

Select an option

Learn more about clone URLs
Save koichik/1250610 to your computer and use it in GitHub Desktop.
Download ZIP

Revisions

  1. koichik revised this gist Sep 29, 2011. 1 changed file with 2 additions and 3 deletions.
    5 changes: 2 additions & 3 deletions gistfile1.md
    View file
    Original file line number Diff line number Diff line change
    @@ -1,9 +1,8 @@
    # プルリクエストを処理するときのチェックリスト

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

    - パッチは master 用か v0.4 用か？
    -- master にリクエストされても v0.4 に適用できるならそうする。
    - パッチは master 用か v0.4 用か？ (master にリクエストされても v0.4 に適用できるならそうする)
    - パッチを確認するためのブランチを作る。
    - パッチを適用する。
    - ビルドしてテスト。
  2. koichik created this gist Sep 29, 2011.
    28 changes: 28 additions & 0 deletions gistfile1.md
    View file
    Original file line number Diff line number Diff line change
    @@ -0,0 +1,28 @@
    # プルリクエストを処理するときのチェックリスト

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

    - パッチは 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」ってコメントするなど。