プルリクエストがマージできない / Cannot merge pull requests

by 宮城築

Slide 1

Slide 1 text

〜不意にとどく設計見直し必須級の指摘との向き合い方〜プルリクエストがマージできない！！ミヤギ

Slide 2

Slide 2 text

特徴・趣味・えんさがそっ♪ を開発中・沖縄からフルリモートで参画・PM に挑戦中・BUMP OF CHICKEN が好き自己紹介宮城　築ミヤギ＠3歳児の父エンジニア｜BABY JOB @KizuMiyagi

Slide 3

Slide 3 text

施設を探す見学の申込園の魅力を発信見学の受付サービス紹介（えんさがそっ♪）あとでちょっと出てくるよ

Slide 4

Slide 4 text

突然ですが、質問です

Slide 5

Slide 5 text

プルリクの指摘対応って

Slide 6

Slide 6 text

大変ですか？

Slide 7

Slide 7 text

大変ですよね

Slide 8

Slide 8 text

私のチームでは

Slide 9

Slide 9 text

プルリクエストの指摘は無視しても良い󰢏

Slide 10

Slide 10 text

というルールがあります

Slide 11

Slide 11 text

ルールができるまでの経緯と効果をご紹介します

Slide 12

Slide 12 text

前提

Slide 13

Slide 13 text

普段の開発サイクル 1. 開発手法はスクラム（最近は1週間に2スプリントと高速で回しています） 2. レビューはチーム内で実施 3. たまに不定期でテックリードのレビューが入る

Slide 14

Slide 14 text

普段の開発サイクル 1. 開発手法はスクラム（最近は1週間に2スプリントと高速で回しています） 2. レビューはチーム内で実施 3. たまに不定期でテックリードのレビューが入る　　　　　　　　　　　　　そう、レビュー指摘が不意に届くのです

Slide 15

Slide 15 text

軽微な指摘「その命名あってる？」

Slide 16

Slide 16 text

実際の指摘

Slide 17

Slide 17 text

実際の指摘要約：文法おかしくない？

Slide 18

Slide 18 text

実際の指摘要約：文法おかしくない？ 1

Slide 19

Slide 19 text

作業 1. 指摘されたソースを修正 2. 同様の語順になっている命名が行われていないか確認＆修正作業影響ほぼなし（同日に対応完了）その後の対応

Slide 20

Slide 20 text

ありがたい😊

Slide 21

Slide 21 text

重めの指摘「なんでその仕様にしたの？」

Slide 22

Slide 22 text

実際の指摘

Slide 23

Slide 23 text

実際の指摘

Slide 24

Slide 24 text

実際の指摘

Slide 25

Slide 25 text

実際の指摘

Slide 26

Slide 26 text

実際の指摘要約：どのように考えて仕様を決めたの？

Slide 27

Slide 27 text

実際の指摘要約：どのように考えて仕様を決めたの？ 1 1

Slide 28

Slide 28 text

作業 1. チーム内で改めて議論 a. 指摘された観点で考えを整理 2. テックリードに結論を報告 a. ソース修正なしその後の対応結論 ※議論中

Slide 29

Slide 29 text

作業 1. チーム内で改めて議論 a. 指摘された観点で考えを整理 2. テックリードに結論を報告 a. ソース修正なし作業影響約半日遅延（翌日に対応完了）その後の対応結論 ※議論中

Slide 30

Slide 30 text

議論って楽しい🗣

Slide 31

Slide 31 text

ヤバイ指摘「ドメインの考え方あってる？」

Slide 32

Slide 32 text

実際の指摘

Slide 33

Slide 33 text

1 1 実際の指摘

Slide 34

Slide 34 text

ちょっと補足

Slide 35

Slide 35 text

保育施設保育時間通知先保育理念保育料定員数見学受付方法施設種別お問い合わせ受付方法担当者名

Slide 36

Slide 36 text

保育施設保育時間通知先保育理念保育料定員数見学受付方法施設種別お問い合わせ受付方法担当者名えんさがそっ♪を利用するための情報現実世界の保育施設の情報

Slide 37

Slide 37 text

保育施設保育時間通知先保育理念保育料定員数見学受付方法施設種別お問い合わせ受付方法担当者名えんさがそっ♪を利用するための情報現実世界の保育施設の情報どっちだと思う？

Slide 38

Slide 38 text

作業 1. ドメイン部分を再設計 2. 指摘されたソースを修正 3. 類似箇所をまとめて別課題として起票作業影響約 2 日遅延（二営業日後に対応完了）その後の対応

Slide 39

Slide 39 text

勉強になる！

Slide 40

Slide 40 text

・・・

Slide 41

Slide 41 text

マージできない😭

Slide 42

Slide 42 text

とあるふりかえりにて・・・

Slide 43

Slide 43 text

指摘対応で遅延するの改善したいよねバッファを設ける？設計の品質を上げよう！

Slide 44

Slide 44 text

😎＜ EM 指摘対応って全部必要？

Slide 45

Slide 45 text

4 4 😎＜ EM 指摘対応って全部必要？

Slide 46

Slide 46 text

指摘の種類によって指摘対応の必須 / 任意を判断する ● 必須　：　MUST / SHOULD ● 任意　：　必須以外　（ IMO / FYI / Q ..etc ） ※指摘する側のルールは変更なしプルリクマージの新ルール爆誕

Slide 47

Slide 47 text

えんさがそっ♪の開発で大切にしていること　※優先度順 1. 最低限の品質担保（仕様漏れ・致命的なバグを防ぐ） 2. 納期遵守（最速で価値を届ける） 3. コード品質の維持向上（保守性を保つ・改善する）任意の指摘は技術負債としてリリース後のタスクに積み上げる新ルールの理屈

Slide 48

Slide 48 text

ということで訂正

Slide 49

Slide 49 text

プルリクエストの指摘は無視しても良い󰢏

Slide 50

Slide 50 text

プルリクエストの指摘は無視しても良い󰢏 　　棚上げ

Slide 51

Slide 51 text

まとめ

Slide 52

Slide 52 text

納期遵守率 UP 納期を考慮したうえで取り組む技術負債の選択ができるようになった ※ただし、様々な要因があるため一概にこのルールのおかげとは言えません新ルールの効果 86% 96%

Slide 53

Slide 53 text

タスク計画の難易度UP 技術負債タスクが増えたことでタスク管理が複雑になった一方で、意思を持って技術負債解消タスクを選択できるようになったため技術負債解消を踏まえた中期的な計画を立てられるようになったチームとの相性は良さそう 🙌 新ルールの副作用