2025-05-14_レビューのつらさを減らす工夫

レビューのつらさを減らす工夫 Easing the Pain of Reviews Money Forward Tech LT大会 vol.3 2025/05/14 藤田恭平 / Kyohei Fujita

お話すること / Today’s topic コーディングやドキュメント作成の後に行われるレビュー レビューをつらく感じる方もいるのでは 経験の浅い方や、新しい組織に入ったばかりの方など Review sessions held once the code or docs are written They can feel daunting to some Especially to newcomers or less-experienced teammates 2

自己紹介 / Self Introduction 藤田恭平 Kyohei Fujita 開発者歴 約12年 Software developer for about 12 years Back-end Engineer Kotlin(Server-side) / Quarkus / RabbitMQ / Google Cloud / Kubernetes / PostgreSQL / GraphQL / gRPC etc. 3

4 1. 小さくする 2. 見てほしいものはハッキリと 3. ラベルで温度感を伝える 4. 納得してから変更する 1. Keep changes small 2. Highlight what to review 3. Labels to signal importance 4. Change only when convinced

1. 小さくする / Keep changes small 大きくても1日分の変更量ぐらいまで 巨大レビュー依頼を出されると、見るのも大変、大きな手戻りになりかねない 誰も嬉しくない Keep it to about one day’s worth of changes Massive review requests are painful and invite costly rework Nobody wins 5

2. 見てほしいものはハッキリと / Highlight what to review 見てほしい点、逆に見なくて良い点は明記する 「テストパターンが足りてるか不安です」とか 「まだ動作確認してないので、コードの大きな流れだけ見てください」とか 期待値が合うと、お互いに余計な時間を使わなくて済む Clearly mark what to review—and what to skip e.g. “I’m not sure the test coverage is sufficient.” e.g. “I haven’t run it yet—just check the overall flow.” Aligned expectations save everyone time 6

3. ラベルで温度感を伝える / Labels to signal importance コメントにラベルを付け「全てが対応必須」と受け取られないようにする Label comments so they don’t all feel like “must-fix” items must 対応必須 Required change imo 個人的な意見や提案。必須ではない Personal opinion / suggestion — optional nits 重箱の隅をつつく提案。無視しても良い Minor nitpicks; fine to ignore q 純粋に質問 Just a question 7

4. 納得してから変更する / Change only when convinced 「よく分からないけど言われたから直しました」は禁止 量が多くて大変な時、納期が迫ってる時、ありがち 納得できるまで質問しないと、成長の機会を逃す レビュアーが間違っていることもある “I changed it because they said so” is a no-go Easy to slip into when the comments pile up or deadlines are tight Keep asking until it clicks—that’s how you grow Reviewers can be wrong, too 8

まとめ / Summary 小さく的を絞る 相手への期待値や温度感は明確に伝える 納得できるまでコミュニケーションを取る Keep it small and focused Spell out expectations and importance Communicate until you’re confident 9