=============================================================
45:ちょっとした修正のつもりでコードを際限なく書き換えてしまう
=============================================================

.. maigo:: ちょっと修正、のついでに

    * 後輩W：昨日レビューしてもらったPRなんですが、問題があったのでちょっと修正しました。修正した問題はちょっとなのでレビューなしで大丈夫です。
    * 先輩T：お、了解（どれどれ、チラッと見ておこうかな……ちょっとじゃない、がっつり書き換わってる！！）がっつり書き直されてるんだけど、どのへんが「ちょっと修正」なの？
    * 後輩W：特定の組合せのときだけエラーになる、ちょっとした問題を修正しました。
    * 先輩T：いやそうじゃなくて、コードレビューしたところがあらかた書き換わってるじゃない。
    * 後輩W：直しながら、どうせなら設計変えたほうがいいと気づいたので、ついでに修正しました。

「修正によって直した動作はちょっとしたもの」だとしても、コードを大幅に書き換えているのであれば再度レビューするべきです。
変更した挙動の大小でレビューするかどうかを決めてしまうと、動作が変わらないリファクタリングはレビュー不要、ということになってしまいます。

ベストプラクティス
==================

挙動が変わるなら、レビューしましょう。
挙動が変わらなくても、変更範囲が大きいならレビューしましょう。

.. omission::