指摘・レビュー4 択難易度 応用

指摘「ついでにこのモジュール全体の命名を見直してほしい」への返信は？

正答

1. 命名の見直しは今回の PR ではやりません。
2. わかりました。ついでなので、この PR でモジュールの全ファイルの命名を直します。
3. （返信せず、指摘を放置したままマージを待つ）
4. この PR の範囲を超えるので、別の Issue に切り出して対応してもよいでしょうか。

解説

1 つの PR に目的の違う大きな変更を混ぜると、差分が膨らんでレビューが難しくなり、問題が起きたときに切り戻しにくい。妥当な指摘でも範囲外なら、別の Issue に切り出すことを提案し、相手の合意を取るのが一般的。理由を言わずに断ると、指摘を軽く扱われたと受け取られやすい。放置すると対応する気が無いのか判断できない。

参考: google.github.io

reviewreplyscope