WIP PR やってみた

末並 晃 ( @a_suenami )

自己紹介• 株式会社ファクトリアル

• Web系受託プログラマ

• RubyとかPHPとか（最近Scalaに興味）

• 好きなもの

• テスト

• オブジェクト指向

コードレビューとしての Pull Request

• トピックブランチを統括ブランチ（通常はmaster）へマージするときには Pull Request を出す

• Pull Request はプロジェクトメンバー全員でレビューし、修正の必要があれば Pull Request に対してコミットを積む

• プロジェクトメンバーがほとんどのコードに目を通すため、仕様誤認、実装不備等を未然に防ぐ

• コードレビューを通してナレッジの平滑化を促進する

WIP PRとは• Work In Progress Pull Request

• GitHubでホスティングされているリポジトリに対する修正について、作業途中の状態でPull Requestを送ること

• Pull Request のタイトルを [WIP] からはじめてレビュワーに作業途中であることを明示する

• Pull Request の本文には残っている TODO を記載する

何が変わった？• 従来よりさらに早いタイミングでコードレビューを実施できるため、修正コストが低減した

• 指摘をすることに対する敷居が下がった

• 課題管理システムとしてのPull Request

• 作業の進捗が透明化される

結論

WIP PR いいよ！！