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 いいよ!!

Wip prをやってみた