コードレビューは作業結果のダブルチェックではない
こうする
https://scrapbox.io/files/6a25e9f6601c8b4efdcf867a.png
プロセスやロジックをダブルチェックするshokai.icon
正気を疑うという事です
こうではない
https://scrapbox.io/files/6a25e9f9601c8b4efdcf867c.png
レビューというと完成物をダブルチェックする行為を思い浮かべがちだが、そうではない
正気を疑うとは
プロセスや考え方をチェックする
現状認識・分析
この機能が何を目的とした機能で、どういう挙動をするのかを軽く説明する
bugの検証方法、このbugからどういう教訓を得たのか
bug修正ではなく新機能の場合も、これが重要
どこを見ているかをチェックする
どういう影響範囲があると考えているか
どういうソフトウェアのコンポーネントに影響があるか
どういう操作に影響があるか
どういうユーザーに影響があるか
ソフトウェア内の特定の領域の担当を奪い取るのに必要な過程です
そのために、Pull Requestに動作確認方法を書けshokai.icon