【新卒/GitHub初心者向け】コードレビュー時のプルリクテンプレ

社会人7年目のゴリゴリのおっさんですが
新卒からこんなプルリクきたら嬉しいなー、
ってか自分もこれやらないとなーっと思ったので。

また今までSVNだったけど4月からGitHub使うことになった方にも
参考になればと。

目的

• コードレビューにかかる時間的コストをさげて、生産性を上げてこ！
• どこをどう見ていいかわからん！という先輩レビュワーの不満解消！
• 先輩レビュワーに対する意思表示(HRTの精神)！
• 自分がしたことの明示化(伝えないと伝わらない)！
• 何をして何をしてないかの意思表示!

以下テンプレ内容

(git hubのコメントに貼り付けるイメージ)

## レビュー期日

## 目的

## やったこと

## やってないこと

## 〇〇さんに見てほしいところ

## 確認済


お願いします。

以下テンプレを使った例

## レビュー期日
 - 6/23 18:00まで
 
## 目的
リダイレクトするとパラメータが消えるバグの解消

## やったこと
 - [x] def process_g08m11_redirectsにparams(typeなどの)を追加
 - [x] def process_rails_redirectsにparams(typeなど)を追加
 - インデント調整やソース可読性向上のためのリファクタリング

## G08M11さんに見てほしいところ
 - 冗長的な書き方の解消

## 確認済
 - 単体テスト
 - 結合テスト


お願いします。

やってみた結果

• テンプレ使ったら所要時間1分も掛からない。
  (規模にもよるかも。フェーズにもよるかも。)

• 好意的に受け取って頂けてる感ある。

• まだ実施中のためもっと良いやり方やチームにfitしたやり方があるかも。

• PDCA回しながら継続化

• もっと状況によってパターン化させていく。

参考記事