状況
PR を見ていたら、自分なら別の書き方をする箇所があった。ただし動作や読みやすさに問題はなく、チームのスタイルガイドにも沿っている。
自分の好みと違う書き方は、修正必須として差し戻すべきである。
正答
× 誤り
解説
スタイルガイドに沿っていて、正しさや読みやすさに問題がない書き方は、作成者の選択を尊重するのが一般的である。好みだけを理由に差し戻すと、レビューが遅くなり、作成者との関係も悪くなる。どうしても伝えたい場合は、「Nit:」などを付けて任意の提案であることを明示する。根拠のある指摘と好みを区別することが、レビューの質を保つ。
参考: google.github.io
この問題を含む 実務シチュエーション のテストを制限時間つきで解く
実務シチュエーションクイズへ