状況
アプリの修正とインフラ設定(Terraform)の変更を含む PR のレビューを頼まれた。アプリ部分は理解できるが、インフラ部分は詳しくない。
レビュアーとして適切な対応は?
正答
- 分からない部分も含めて、全体を Approve する
- 分からない部分があるので、レビュー全体を断る
- 確認した範囲を明記し、インフラ部分は詳しい人にも見てもらう
- インフラ部分は作成者を信じて、確認済みとコメントする
解説
Approve は「この変更を確認した」という意思表示なので、理解していない部分まで承認すると、誰も確認していない変更が通ってしまう。自分が確認できた範囲を明記してレビューし、詳しくない部分は適任者にレビューを依頼するのが適切。分かる部分まで含めて断ると、確認できたはずの観点も失われる。確認していないのに確認済みと書くのは、記録として誤りである。
この問題を含む 実務シチュエーション のテストを制限時間つきで解く
実務シチュエーションクイズへ