どのコミットで修正するべき？

Question

過去のコミットに戻って作業ができるを知り、ふと疑問が湧きました
Aの関数にバグが見つかったとします。
Aを関数を作成したコミットに戻って修正するのか、新しいコミットで修正するほうがいいのか。
どちらでしょうか？

Comments

[tempra_o]

Aの関数作成時だと、そのコミットでAの関数については作成、修正が完結して見やすいと思いました。ただ、修正した履歴としてはどうなのかな？時系列の方がいいのではないのかなと？gitの考え方がいまいちわかっていません。

Answer 1

新しいコミットをするべきです。

過去のコミットに戻った場合、そのコミットから先の修正がなかったことになります。つまり、それ以降のコミットを再び実施することになります。
また、それが最新の修正であっても、他の使用者がそのコミットを取り込んでいた場合、その人のリポジトリが壊れてしまい、最初から作り直しになってしまいます。

つまり、過去のコミットに戻ることはデメリットが大きいということです。

リポジトリ以外にも問題があります。
何か問題があった際、いつどのようにしてその不具合が紛れ込んだのかがわからなくなるということです。
開発を続けていくと、最初は正常に動いていたものが途中で動かなくなる、なんてことは往々にしてあります。
その場合、過去のコミットを見直すことで原因となる箇所、それが入った時期などを見つけることが可能なのですが、戻ってコミットなどをしていると、結局それがいつ紛れ込んだのか、正常なタイミングが把握できなくなります。

こんな状況になってしまってはバージョン管理ツールを使う意味がなくなってしまいます。

ですので、基本的には過去に戻ってコミットをし直すことはしてはいけないことだと思っています。