ボーイスカウト・ルール、先に直すか、後から直すか。 - ちなみに

ボーイスカウト・ルール、先に直すか、後から直すか。 - ちなみに

ボーイスカウトルール。 機能追加より、 先に直すか、 後から直すか。— 人生踊れば丸儲け (@tomohi_ro) ...概要を表示 ボーイスカウトルール。 機能追加より、 先に直すか、 後から直すか。— 人生踊れば丸儲け (@tomohi_ro) 2022年4月3日 まれによく目的の変更にリファクタリングを混ぜたプルリクのレビュー依頼を受けるので、そのたびに変更とリファクタリングは分けてねって話をしている。 レビューするときにもdiffが混ざっていて見にくいし、将来的に変更の経緯を追いかけるときにもノイズになってしまう。 コミットが分かれていたらまだマシなんだけれど、そういう場合はたいてい同じコミットに複数の目的が混ざってしまっている。 www.clear-code.com 僕は1コミットには1つのことというルールを出来るだけ守ろうと思って暮らしているのでどちらかというと過激派なのですが、プルリク単位くらいではやることを絞って欲しいとは思っています。 みんな単一責任原則は大好きな気がするけど、コミットやプルリクにも同じ

• 開発