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

Transcript

1. 〜 不意にとどく設計見直し必須級の指摘との向き合い方 〜 プルリクエスト が マージ できない！！ ミヤギ
2. 特徴・趣味 ・えんさがそっ♪ を開発中 ・沖縄からフルリモートで参画 ・PM に挑戦中 ・BUMP OF CHICKEN が好き 自己紹介 宮城　築 ミヤギ＠3歳児の父エンジニア｜BABY JOB @KizuMiyagi
3. 施設を 探す 見学の 申込 園の魅力 を発信 見学の 受付 サービス紹介 （えんさがそっ♪） あとでちょっと 出てくるよ
4. 突然ですが、質問です
5. プルリクの指摘対応って
6. 大変ですか？
7. 大変ですよね
8. 私のチームでは
9. プルリクエストの指摘は 無視 しても良い󰢏
10. というルールがあります
11. ルールができるまでの 経緯と効果をご紹介します
12. 前提
13. 普段の開発サイクル 1. 開発手法は スクラム （最近は1週間に2スプリントと高速で回しています） 2. レビューは チーム内 で実施 3. たまに不定期で テックリード のレビューが入る
14. 普段の開発サイクル 1. 開発手法は スクラム （最近は1週間に2スプリントと高速で回しています） 2. レビューは チーム内 で実施 3. たまに不定期で テックリード のレビューが入る 　　　　　　　　　　　　　 そう、レビュー指摘が不意に届くのです
15. 軽微な指摘 「その命名あってる？」
16. 実際の指摘
17. 実際の指摘 要約 ： 文法おかしくない？
18. 実際の指摘 要約 ： 文法おかしくない？ 1
19. 作業 1. 指摘されたソースを修正 2. 同様の語順になっている命名が行われていないか確認＆修正 作業影響 ほぼなし （同日に対応完了） その後の対応
20. ありがたい😊
21. 重めの指摘 「なんでその仕様にしたの？」
22. 実際の指摘
23. 実際の指摘
24. 実際の指摘
25. 実際の指摘
26. 実際の指摘 要約 ： どのように考えて仕様を決めたの？
27. 実際の指摘 要約 ： どのように考えて仕様を決めたの？ 1 1
28. 作業 1. チーム内で改めて議論 a. 指摘された観点で考えを整理 2. テックリードに結論を報告 a. ソース修正なし その後の対応 結論 ※議論中
29. 作業 1. チーム内で改めて議論 a. 指摘された観点で考えを整理 2. テックリードに結論を報告 a. ソース修正なし 作業影響 約 半日 遅延 （翌日に対応完了） その後の対応 結論 ※議論中
30. 議論って楽しい🗣
31. ヤバイ指摘 「ドメインの考え方あってる？」
32. 実際の指摘
33. 1 1 実際の指摘
34. ちょっと補足
35. 保育施設 保育時間 通知先 保育理念 保育料 定員数 見学受付方法 施設種別 お問い合わせ 受付方法 担当者名
36. 保育施設 保育時間 通知先 保育理念 保育料 定員数 見学受付方法 施設種別 お問い合わせ 受付方法 担当者名 えんさがそっ♪を利 用するための情報 現実世界の 保育施設の情報
37. 保育施設 保育時間 通知先 保育理念 保育料 定員数 見学受付方法 施設種別 お問い合わせ 受付方法 担当者名 えんさがそっ♪を利 用するための情報 現実世界の 保育施設の情報 どっちだと思う？
38. 作業 1. ドメイン部分を再設計 2. 指摘されたソースを修正 3. 類似箇所をまとめて別課題として起票 作業影響 約 2 日 遅延 （二営業日後に対応完了） その後の対応
39. 勉強になる！
40. ・・・
41. マージできない😭
42. とあるふりかえりにて・・・
43. 指摘対応で遅延するの改善したいよね バッファを設ける？ 設計の品質を上げよう！
44. 😎＜ EM 指摘対応って 全部必要？
45. 4 4 😎＜ EM 指摘対応って 全部必要？
46. 指摘の種類によって指摘対応の 必須 / 任意 を判断する • 必須　：　MUST / SHOULD • 任意　：　必須以外　（ IMO / FYI / Q ..etc ） ※指摘する側のルールは変更なし プルリクマージの新ルール爆誕
47. えんさがそっ♪の開発で大切にしていること　※優先度順 1. 最低限の品質担保 （仕様漏れ・致命的なバグを防ぐ） 2. 納期遵守 （最速で価値を届ける） 3. コード品質の維持向上 （保守性を保つ・改善する） 任意の指摘は 技術負債 としてリリース後のタスクに積み上げる 新ルールの理屈
48. ということで訂正
49. プルリクエストの指摘は 無視 しても良い󰢏
50. プルリクエストの指摘は 無視 しても良い󰢏 　　棚上げ
51. まとめ
52. 納期遵守率 UP 納期を考慮したうえで取り組む技術負債の選択ができるようになった ※ただし、様々な要因があるため一概にこのルールのおかげとは言えません 新ルールの効果 86% 96%
53. タスク計画の難易度UP 技術負債タスクが増えたことで タスク管理が複雑 になった 一方で、意思を持って技術負債解消タスクを選択できるようになったため 技術負債解消を踏まえた 中期的な計画 を立てられるようになった チームとの相性は良さそう 🙌 新ルールの副作用
54. つまり、副作用については
55. PM の腕の見せどころ💪 \ ﾔｯﾃﾔﾙ /
56. ご清聴ありがとうございました