code review とは何か？
code review とは、変更が共有コードベースに merge される前に、1 人以上の同僚が提案された変更を精査するプラクティスです。
code review は、人間の目が CI の自動チェックを補完する場です。変更がメインコードベースの一部になる前に、チームメイトがそれを読み、質問し、フィードバックを与えます。ツールには捉えられない問題を捉え、チーム全体に知識を広めます。
何であるか
code review では、作者が通常 pull request として変更を提案し、reviewer が diff を読み、正しさ、明瞭さ、設計を検討し、承認するか変更を要求します。変更が merge される前に行われる、意図的な品質ゲートです。
プロセスの仕組み
- 作者が変更を説明する pull request を開く。
- CI が自動チェックを実行する: build、test、linting。
- reviewer が diff を読みコメントを残す。
- 作者がフォローアップの commit でフィードバックに対応する。
- 承認されチェックが合格したら、変更が merge される。
例
開発者が caching レイヤーを追加する pull request を開きます。CI はテストが合格することを確認し、reviewer はデータが変わったときに cache が決して無効化されないことに気づきます。どのテストもカバーしていなかった微妙なバグです。作者が修正し、reviewer が承認し、将来の production インシデントが回避されます。
なぜ review が品質を向上させるか
reviewer は、自動チェックが見逃すロジックエラー、不明瞭なコード、欠けたテスト、設計上の問題を捉えます。同じくらい重要なことに、review は理解を広めます。今や 2 人以上がその変更の仕組みを知っています。読まれると分かっているので、作者をより明瞭なコードへ促す効果もあります。
CI とうまく連携する
自動チェックと人間の review は労力を分担します。CI は機械的で疲れを知らない作業、build できるか、テストは合格するか、フォーマットされているか、を担い、reviewer は判断に集中できます。これは正しい設計か、正しいか、保守可能か。両者が合わさって merge 前の強力なゲートを形成します。
重要なポイント
- code review は merge の前に同僚が変更を精査する。
- ツールが見逃す問題を捉え、チームの知識を広める。
- CI と組み合わさる。機械が仕組みをチェックし、人間が設計を判断する。