プルリクエストとコードレビューの演劇
2025-09-25
Goatmire Elixir ConfでのSaša Jurić氏の講演は、コードレビューを魅力的な物語に変えました。彼は、大きくて複雑なプルリクエスト(PR)の一般的な問題点を指摘し、それが表面的なレビュー、セキュリティリスク、保守困難なコードベースにつながることを強調しました。重要なのは、レビュー可能なPRは理想的には5~10分でレビューできること、そしてコード行数が300行以下であることです。これは、各変更の理由と手順を明確に説明する簡潔で物語的なコミットメッセージを作成することで実現します。大きな機能を小さなPRに分割し、`git fixup`などのツールを使用してクリーンなコミット履歴を維持することは、効率的なコードレビューとより高品質なコードにとって不可欠です。講演では、「理解できません」と言う方が意味のない「LGTM」よりも優れていると強調されました。
開発