【チーム開発】プルリクを分割して レビュー負荷を軽減する

チーム開発の負荷軽減として、プルリクの分割が注目されています。
本スライドでは、どのようにしてプルリクを分割すればよいのか？具体的な方法を解説していきます。
また、プルリクを分割するメリットを説明することで導入イメージを深めていきます

Transcript

1. プルリクを分割して レビュー負荷を軽減する
2. 自己紹介 エンジニア3年目 趣味：サウナ、ボードゲーム 　ぎゅう（@gyu_outputs） 　→ 前職:Laravel / Vue / AWS 　→ Golangを現在メイン 　→ エンジニア仲間と行きます
3. 技術本を友達と書いてます
4. レビューが大変という 経験はございませんか？
5. 変 更 フ ァ イ ル 数 多 い な ぁ よ し 、 あ と で 見 る か ぁ 分割でレビューが必要になる
6. 前どこまで確認したっけ？ ファイル数が多いと起こりがち...。
7. ま だ レ ビ ュ ー 中 だ か ら 、 も う 少 し 待 っ て ね レビューが後回しにされることも
8. 一気にマージされて、コンフリクトに苦しむ コ ン フ リ ク ト 多 す ぎ こ れ 全 部 修 正 は 大 変 だ っ て
9. そんな現場に役立つ方法
10. プルリクを分割して レビュー負荷を軽減する
11. プルリク分割が注目されている
12. 現職でもかなり分割されている そ う い え ば 当 た り 前 で は な い の か ？ こ の 会 社 の リ リ ー ス 回 数 多 く て 驚 き ま し た ※ フ リ ー ラ ン ス
13. 変更行が500行は多すぎです プルリク分割してください 自分も過去に注意をされる
14. でも、どう分割すれば？
15. そんな話をします
16. プルリクの分割単位
17. findyの資料を見てみましょう
18. ※findyより
19. 補足説明を入れると、
20. ※findyの画像に補足説明を追加 テーブル作成 モデル作成 Repository作成 usecase作成 handler作成
21. 利用していないから、リリースしても問題なし テーブル作成 モデル作成 Repository作成 usecase作成 handler作成 リリースしても影響なし QA実施
22. テーブルを作成 model, Entityを作成 handlerを空で作成。swaggerを記述してAppチームに共有 必要であれば検証用に仮データをreturn Repositoryの実装 Usecaseの実装 handlerの中身を実装 フロントエンドとAPIの繋ぎ込み→ 動作確認 → QA実施 関数が必要になったら、こまめにプルリク発行 1. 2. 3. a. 4. 5. 6. 7. 8. プルリク分割の単位
23. 完 全 に 理 解 し た
24. モ デ ル に 関 数 追 加 し た し 、 テ ス ト 書 い て プ ル リ ク あ げ よ
25. LGTM 2分ほどでプルリクを確認できる
26. レビュー時間を圧倒的に短縮
27. メリットはそれだけではない
28. メリット① コンフリクトの負荷軽減
29. handler usecase Repository handler usecase Repository API API コンフリクト コンフリクト コンフリクト マージ 機能が一つ増えると、コンフリクトの影響範囲が大 一気に修正は大変
30. API handler usecase Repository API handler usecase Repository コンフリクト コンフリクト コンフリクト コンフリクトが分割されることで、負担が軽減される
31. コンフリクトのストレス軽減
32. マイクロサービスアーキテクチャ などを活用しなくても 問題なく開発が進められる
33. メリット② タスク管理がしやすい
34. handler usecase Repository: Get TODO TODO DOING Repository: Save TODO タスクが細分化されて、状況がわかりやすい 早くしないと...。
35. handler usecase Repository: Get DOING DOING DOING Repository: Save DOING フォローしやすい 手伝いますね 手伝いますね 手伝いますね ありがとうございます
36. 状況に合わせて 柔軟に対応できる
37. 予定より早くリリースできました 競合他社よりも早くリリースできる
38. モブプロで設計を決める
39. このコード何？ってなるのでは？
40. モブプロで実装方針を整理する Notionで実装方針の認識を合わせる
41. あの箇所の プルリクですね
42. 実装方針の認識が一緒なら 問題なし
43. Githubでなるべ く管理したい モブプロの最適解は模索中
44. 最後に、、、
45. 負担が軽減することで 毎日2~5回リリースも可能
46. あなたの会社でも プルリクを分割しては？
47. ご清聴ありがとうございました