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

Slide 1

Slide 1 text

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

Slide 2

Slide 2 text

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

Slide 3

Slide 3 text

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

Slide 4

Slide 4 text

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

Slide 5

Slide 5 text

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

Slide 6

Slide 6 text

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

Slide 7

Slide 7 text

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

Slide 8

Slide 8 text

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

Slide 9

Slide 9 text

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

Slide 10

Slide 10 text

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

Slide 11

Slide 11 text

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

Slide 12

Slide 12 text

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

Slide 13

Slide 13 text

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

Slide 14

Slide 14 text

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

Slide 15

Slide 15 text

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

Slide 16

Slide 16 text

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

Slide 17

Slide 17 text

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

Slide 18

Slide 18 text

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

Slide 19

Slide 19 text

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

Slide 20

Slide 20 text

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

Slide 21

Slide 21 text

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

Slide 22

Slide 22 text

CONFIDENTIAL 22 前提：導入前の状況や課題  ① チームの健康状態を可視化したい  チームの目標値や振り返りのきっかけにも ②レビュー難易度が高く気乗りしない  それぞれの変更がどう関連しているのかがわからない...

Slide 23

Slide 23 text

CONFIDENTIAL       Findy Team+の導入  01   PR分割の提案  02 チーム課題の解決策  23 導入のきっかけ  とりあえずやってみよう精神で  すんなりと取り組み開始  

Slide 24

Slide 24 text

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

Slide 25

Slide 25 text

CONFIDENTIAL 25 方法：PR分割の基準  普段使っている  PRのテンプレート

Slide 26

Slide 26 text

CONFIDENTIAL 26 方法：Findy Team+の使い所や用途    分析用のデータ活用  01   アウトプットや  健康状態の指標  02 1on1 スクラム  イベント

Slide 27

Slide 27 text

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

Slide 28

Slide 28 text

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

Slide 29

Slide 29 text

CONFIDENTIAL 障壁：PR数増加によるレビュー負担  29 ● レビュアーの偏り  ● 作業時間の減少  ● PR作成の頻度増影響 ● PRレビューの担当制  ○ 曜日毎  ○ ストーリー毎  ● リーダーはレビュー必須  ※2名approve必須のため  ● PR作成時の各種自動化  ○ ラベルやレビュアーを GithubActionsで自動付与対策 FindyTeam+  から把握

Slide 30

Slide 30 text

CONFIDENTIAL 障壁：その他課題  30 GithubActionsの  使用時間が爆増レビュー完了していないPRどれ？事業に対する生産性が上がっているかは不明

Slide 31

Slide 31 text

CONFIDENTIAL ちょっと小話  31 月曜日休み火曜日出勤したら.... レビュー待ち  PR数が...12!?  不思議と嬉しい...♪  さぁウォーミングアップやっ！ 

Slide 32

Slide 32 text

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

Slide 33

Slide 33 text

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

Slide 34

Slide 34 text

CONFIDENTIAL 34 まとめ：PR分割のメリット・デメリット  ● 開発サイクルの向上  ● レビューの負担軽減  ● 手戻りの抑止力UP  ● アウトプット増  ● 計画調整の容易性UP  メリット ● レビュイーの負担微増  ● PRの管理が煩雑化   ※プロジェクト規模や人数に比例     ● ゴールイメージが共有しにくい（レビュアー目線）  デメリット

Slide 35

Slide 35 text

CONFIDENTIAL 35 まとめ  PR分割をぜひ皆さんのチームに導入してほしい。  きっと導入後の世界に入りたくなっているはずです。 

Slide 36

Slide 36 text

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