16:01:57 @AmaiSaeta@oransns.com
icon

何故、皆、特定のpull requestに、それで実現したい事以外の変更(リファクタとか、後にやりたい大規模改修の為の布石とか、既存コードのtypo修正とかバグ潰しとか……)を入れたがるの……

16:07:52 @AmaiSaeta@oransns.com
icon

それ、主題となる改修以外を行うコストも、それをレビューするコストも余計に掛かるって事だよ?
「主題の方は問題ないけど、ついでにくっついてる変更が拙いからレビュー通らない」とか、なり得るわけだけど、それで良いの?

16:17:53 @AmaiSaeta@oransns.com
icon

あんまり原理原則に拘っても上手く回らないだろうから、コメント追加とか、処理の一部をprivateメソッドに切り分けるとか程度なら許容するけど、変に影響デカイ事は止めようよ……