社会人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回しながら継続化
もっと状況によってパターン化させていく。