実務シチュエーション4 択難易度 基礎

状況

あなたは、チームメンバーが出したプルリクエスト(PR)のレビュアーに指名された。レビューに使える時間は限られている。

最初に優先して確認すべき観点は?

正答

  1. 変数名やインデントが自分の好みと合っているか
  2. 変更が仕様どおりに動き、他の機能を壊さないか
  3. コミットメッセージの英語表現が自然かどうか
  4. より新しいライブラリに置き換えられないか

解説

レビューで最も重要なのは、変更が意図どおりに正しく動くか、既存の機能やデータに悪影響を与えないかである。時間が限られているときほど、この観点を先に見る。命名や書式は自動整形やスタイルガイドで解決できることが多く、好みの違いは優先度が低い。ライブラリの置き換えは PR の目的から外れた話であり、別の議題として扱うのが適切。

参考: google.github.io

review

この問題を含む 実務シチュエーション のテストを制限時間つきで解く

実務シチュエーションクイズへ