やめられない、とまらない！プルリクエスト分割

■ 2023/04/26（水）開催
　
■ 【開発生産性 Meetup #1】開発生産性可視化による変化~事例LTから学ぶベストプラクティス~
https://findy.connpass.com/event/279894/

Findy主催のMeetupにて登壇したLT会資料になります。

Transcript

1. CONFIDENTIAL 1 やめられない、とまらない！
 プルリクエスト分割
 株式会社うるる 後藤大樹

2. CONFIDENTIAL 【名前】
 後藤 大樹 (ごとう だいき)
 
 
 【所属】
 NJSS事業本部
 スクレイパー開発チーム リーダー（※3月まで）
 
 【経歴】
 • 29歳からエンジニアキャリアスタートして、5年目突入
 • バックエンドメイン、インフラちょこっと。
 2 自己紹介

3. CONFIDENTIAL 3 会社紹介 - 株式会社うるる
 全国の入札情報を一括検 索・管理出来るSaaS型の 業務支援サービス
 入札情報の自動収集シス テムを担当

4. CONFIDENTIAL 4 本日話すこと
 PR分割を導入し、開発の世界がどう変わったのか
 というのをプレイヤー目線も含めてお話します

5. CONFIDENTIAL 5 アジェンダ
 01 今日一番伝えたいこと
 02 その根拠はなにか？
 03 実際にどうやって導入・運用したの?
 04 導入後の課題は？どう解決したの？
 05 まとめ

6. CONFIDENTIAL 6 アジェンダ
 01 今日一番伝えたいこと
 02 その根拠はなにか？
 03 実際にどうやって導入・運用したの?
 04 導入後の課題は？どう解決したの？
 05 まとめ

7. CONFIDENTIAL 7 今日一番伝えたいこと
 PR分割をぜひ皆さんのチームに導入してほしい。
 きっと導入前の世界には戻れなくなるはずです。
 少なくとも私は二度と戻りたくないです。

8. CONFIDENTIAL 8 アジェンダ
 01 今日一番伝えたいこと
 02 その根拠はなにか？
 03 実際にどうやって導入・運用したの?
 04 導入後の課題は？どう解決したの？
 05 まとめ

9. CONFIDENTIAL 9 その根拠はなにか？......の前に
 皆さんこんな世界にいたこと、
 もしくはいる人はいませんか？
 ※すべてフィクションです

10. CONFIDENTIAL 10 その根拠はなにか？......の前に
 PRレビューお願いします！
 はーい、見ますよ〜！

11. CONFIDENTIAL 11 その根拠はなにか？......の前に
 ん...? 変更ファイル数...1...20........120!?
 PRの説明文...長いなぁ....
 そもそもビルドもテストも落ちてるやないかいっ！

12. CONFIDENTIAL 12 その根拠はなにか？......の前に
 あとで見よっ♪

13. CONFIDENTIAL 13 その根拠はなにか？......の前に
 どの仕様変更がどの差分に関連しているんだろう？
 デカイPRは時間かかるんだよな〜......
 そういえば...
 ウルルさんのPRはいつ出てくるんだろう？

14. CONFIDENTIAL 14 その根拠はなにか？......の前に
 レビュー時間の無駄 ♪ 
 作業時間の無駄♪ 
 これなら俺が作れば 
 よかったよ♪ 
 今後も来るのかデカPR♪ 
 それなら俺はもう見ねぇ♪ 

15. CONFIDENTIAL 15 その根拠はなにか？......の前に
 PRレビューが来ると「あとで見よ」
 という心理状態になっていませんか？

16. CONFIDENTIAL 16 その根拠はなにか？...本題
 実際にPR分割を導入後、どう世界が変わったのか

17. CONFIDENTIAL 17 定性的な観点
 進捗状況が明確となり、
 計画の調整がしやすい PRレビューを真っ先に見るよ うに。しかも苦じゃない 実装中の迷いや悩み、
 手戻りが減少
 お祭り感が出て盛り上がる

18. CONFIDENTIAL 18 定量的な観点
 PR分割導入後
 PR数平均：14
 PR数平均：6
 2.3倍
 PRオープンからマージまでが 4分の1まで短縮
 ※ 参照：Findy Team+ 

19. CONFIDENTIAL 19 定量的な観点
 前半と後半を比較
 稼働日単位での平均 

20. CONFIDENTIAL 20 アジェンダ
 01 今日一番伝えたいこと
 02 その根拠はなにか？
 03 実際にどうやって導入・運用したの?
 04 導入後の課題は？どう解決したの？
 05 まとめ

21. CONFIDENTIAL 21 前提：プロジェクト&チームの概要
 • スクレイパーのリプレイスプロジェクト 
 • 新規開発が一旦完了し、運用保守の フェーズ突入
 プロジェクト
 • 4人チーム（全員バックエンド）
 • スクラム開発（振り返りの習慣あり） 
 • 基本オンライン
 チーム
 Backend
 Leader/
 Backend
 Backend
 Backend

22. CONFIDENTIAL 22 前提：導入前の状況や課題
 ① チームの健康状態を可視化したい
 チームの目標値や振り返り のきっかけにも ②レビュー難易度が高く気乗りしない
 それぞれの変更がどう関 連しているのかがわからな い...
23. CONFIDENTIAL 
 
 
 Findy Team+の導入
 01 
 PR分割の提案
 02 チーム課題の解決策
 23 導入のきっかけ
 とりあえずやってみよう精神で
 すんなりと取り組み開始 

24. CONFIDENTIAL 24 方法：PR分割の基準
 • PRの差分は基本1つの機能単位や処 理に絞る（単一責任の原則）
 
 • PRの概要に下記を記載
 ◦ WHAT、HOW、WHYの記載
 ◦ 分割TODOの記載
 レビュイー
 ※PR作成者
 • なる早で見る
 ※レビュイーが頑張っているので敬意を持って 
 
 • LGTMの画像選別
 ※割と大事
 レビュアー
 ※PRレビュー者 

25. CONFIDENTIAL 25 方法：PR分割の基準
 普段使っている
 PRのテンプレート
26. CONFIDENTIAL 26 方法：Findy Team+の使い所や用途
 
 分析用のデータ活用
 01 
 アウトプットや
 健康状態の指標
 02 1on1 スクラム
 イベント
27. CONFIDENTIAL 27 アジェンダ
 01 今日一番伝えたいこと
 02 その根拠はなにか？
 03 実際にどうやって導入・運用したの?
 04 導入後の課題は？どう解決したの？
 05 まとめ

28. CONFIDENTIAL 障壁：どんな課題があったのか
 28 運用していく中でぶつかった障壁 PR数増加によるレビュー負担

29. CONFIDENTIAL 障壁：PR数増加によるレビュー負担
 29 • レビュアーの偏り
 • 作業時間の減少
 • PR作成の頻度増 影響 • PRレビューの担当制
 ◦ 曜日毎
 ◦ ストーリー毎
 • リーダーはレビュー必須
 ※2名approve必須のため
 • PR作成時の各種自動化
 ◦ ラベルやレビュアーを GithubActionsで自動付与 対策 FindyTeam+
 から把握
30. CONFIDENTIAL 障壁：その他課題
 30 GithubActionsの
 使用時間が爆増 レビュー完了して いないPRどれ？ 事業に対する生産 性が上がっている かは不明
31. CONFIDENTIAL ちょっと小話
 31 月曜日休み 火曜日出勤したら.... レビュー待ち
 PR数が...12!?
 不思議と嬉しい...♪
 さぁウォーミングアップ やっ！

32. CONFIDENTIAL 32 アジェンダ
 01 今日一番伝えたいこと
 02 その根拠はなにか？
 03 実際にどうやって導入・運用したの?
 04 導入後の課題は？どう解決したの？
 05 まとめ

33. CONFIDENTIAL 33 まとめ：PR分割導入の前提条件
 PR分割 
 疎結合な
 設計
 設計
 コミュニケーションが活 発になるためお互いの 理解が必要 開発プロセスが 自動化
 されており、PR作成時に品 質が最低限担保 されている 状態が必要 
 PR同士が干渉しないよ うにコンポーネント間の 依存関係を少なくする 必要がある 
 
 CI/CD
 の導入
 自動化
 
 レビューと 改善の文化
 チーム

34. CONFIDENTIAL 34 まとめ：PR分割のメリット・デメリット
 • 開発サイクルの向上
 • レビューの負担軽減
 • 手戻りの抑止力UP
 • アウトプット増
 • 計画調整の容易性UP
 メリット • レビュイーの負担微増
 • PRの管理が煩雑化 
 ※プロジェクト規模や人数に比例 
 
 • ゴールイメージが共有しにくい （レビュアー目線）
 デメリット
35. CONFIDENTIAL 35 まとめ
 PR分割をぜひ皆さんのチームに導入してほしい。
 きっと導入後の世界に入りたくなっているはずです。

36. CONFIDENTIAL 36 ご清聴ありがとうございました。
 株式会社うるる 後藤大樹