プルリクエストの粒度

## 参考資料
Best Practices for Code Review: https://smartbear.com/learn/code-review/best-practices-for-peer-code-review/

Transcript

1. PRの粒度は最小単位やで みんな幸せになれるPR（プルリク）の心得
2. PRで考えることはたくさんあるけど、 ここでは「粒度」にフォーカスします。
3. そもそもPR（プルリクエスト）って何ですか？ PRとはソフトウェア開発において、ある人が行ったコードの変更を、他の人がレビュー し、最終的にコードベースに統合するためのリクエストのこと。 Pull Request Merge Delivery エンジニア レビュー担当者 サーバー ユーザー
4. 建築現場に例えてみます 大工（レビュイー） 現場監督（レビュアー） 家を増築したくて2階建に したいんです。
5. よくあるアンチパターン PR出します 一旦全部組み立てしてか ら持ってこう。 確認項目多すぎて 細かい品質確認は 無理やで！ ## 申し送り事項 ・窓を2つつけました ・ドアをつけました ・etc. 大工（レビュイー） 現場監督（レビュアー）
6. みんな幸せなパターン 大工（レビュイー） 現場監督（レビュアー） パーツ毎に小分けにして 持って行こう ## 窓を作りました ・素材はA社製です ・開け閉めしやすく〇〇な加 工してます PR出します 細かいレビューがし やすくて安心
7. どれくらいの粒度に抑えるべきか 400行を超える行数のコードを一 度にレビューすると不具合の検 知数がかなり低くなる。 大きくても200~400行に納められ るのが理想。 Best Practices for Code Review #Review fewer than 400 lines of code at a time | SmartBear
8. まとめ • 今から行うタスクがビジネスオーダーに近いままなら要注意 • PRのコードサイズは400行が限界 • PRを最小単位にすると ◦ FB時間を短縮できる ◦ レビュアーの負荷を軽減できる ◦ 不具合検知率が高くなる