状況
あなたは、チームメンバーが出したプルリクエスト(PR)のレビュアーに指名された。レビューに使える時間は限られている。
最初に優先して確認すべき観点は?
正答
- 変数名やインデントが自分の好みと合っているか
- 変更が仕様どおりに動き、他の機能を壊さないか
- コミットメッセージの英語表現が自然かどうか
- より新しいライブラリに置き換えられないか
解説
レビューで最も重要なのは、変更が意図どおりに正しく動くか、既存の機能やデータに悪影響を与えないかである。時間が限られているときほど、この観点を先に見る。命名や書式は自動整形やスタイルガイドで解決できることが多く、好みの違いは優先度が低い。ライブラリの置き換えは PR の目的から外れた話であり、別の議題として扱うのが適切。
参考: google.github.io
この問題を含む 実務シチュエーション のテストを制限時間つきで解く
実務シチュエーションクイズへ