優しいPR文化を広め、1ヶ月250件 爆速でレビューを回した話

レビュアーの負担を軽減する「優しいPR文化」をチームに浸透させることで、
1ヶ月に250件という驚異的なペースでレビューを回すことに成功しました。
レビュー時間やマージまでの時間を大幅に短縮した具体的な工夫を紹介します。

Transcript

1. 優しいPR⽂化を広め、 1ヶ⽉250件 爆速でレビューを回した話 パイオニア株式会社 Cross Technology Centerサービス事業本部サービス開発部 ⼩沢 拓也
2. ⼩沢 拓也 2016/04〜2022/03 組み込み開発 2022/04〜 iOS アプリ開発 2024/10〜2025/07 ⼤規模なリファクタリングプロジェクト iOSチームリーダー 2 ⾃⼰紹介
3. リファクタリングのプロジェクトにて iOSリーダーとして数多くのPRをレビュー PR総数：1295件 (レビューしたPR数：966件) ⼀番多い時期で10名体制 レビューしても、次々に新規のPRで作られ終わりの⾒えない⽇々… 3 苦労したこと
4. リファクタリングのプロジェクトにて iOSリーダーとして数多くのPRをレビュー PR総数：1295件 (レビューしたPR数：966件) ⼀番多い時期で10名体制 レビューしても、次々に新規のPRで作られ終わりの⾒えない⽇々… そんな過酷な状況を救ってくれた優しいPRの⽂化を広め 1ヶ⽉250件 爆速でレビューを回した話 4 苦労したこと
5. 5 PRのルール テンプレート 修正⾏数
6. 6 PRのルール これらのルールだけでは、 レビューで苦労するPRが多かった
7. 7 PRのルール これらのルールだけでは、 レビューで苦労するPRが多かった 優しいPRとは？
8. 8 レビュアーに優しいPR 修正理由や影響 修正箇所の仕様書リンク
9. 指摘修正のコミットリンク 9 レビュアーに優しいPR レビューする際のポイント 議論したい箇所
10. 10 レビュアーに優しいPR レビューの流れ 内容の理解 → 修正内容は正しいか？ → 保守性は？
11. 11 レビュアーに優しいPR レビューの流れ 内容の理解 → 修正内容は正しいか？ → 保守性は？ これらを確認する⼿助けになる情報が PRに記載されているとレビューしやすい
12. 12 レビュアーに優しいPR 議事録の抜粋 ・PRのレビューは何を重視しますか？ ⼩さいPRはコードの品質を重視しやすい ⼤きいと動作確認によっていってしまう ・レビューしてて嬉しいのは？ ⾃分でコメント⼊れてる⼈ ネスト変わっただけ(⾒なくていい)等のコメントあるといい 開発メンバー内でPRについて議論
13. 13 効果 議論した タイミング
14. 14 効果 コメントの⽂化や PRの分割が浸透 PRレビューの負荷軽減 議論した タイミング
15. 15 効果 マージまでの時間 も減少 議論した タイミング
16. 16 効果 レビュアーに優しいPR⽂化を広め、 レビューを効率よく⾼速に回すことができた
17. 17 ディスカッションテーマ レビューを依頼するとき、どんな⼯夫をしていますか？ レビューを依頼されたとき、 どんな⼯夫がされているとレビューしやすいですか？
18. 18 ディスカッションテーマ COCCHiアプリ 使ってみてね！ レビューを依頼するとき、どんな⼯夫をしていますか？ レビューを依頼されたとき、 どんな⼯夫がされているとレビューしやすいですか？