実務シチュエーション○×難易度 基礎

完璧なコードでなくても、残っているのが Nit の提案だけなら、提案を添えて Approve してよい。

正答

○ 正しい

解説

正しい。「完璧になるまで Approve しない」はよくある誤解で、レビューの基準は「完璧かどうか」ではなく、「変更によってコードベースの状態が良くなるか」である。正しさに問題がなく、残りが任意の提案だけなら、提案を添えて Approve するのが一般的。Nit が直るまで止めると、価値のある変更の反映が不必要に遅れ、作成者の負担も増える。重要な問題がある場合は、もちろん承認せずに修正を求める。

参考: google.github.io

review