web-dev-qa-db-ja.com

以前のコミットに対して修正を行う場合、別の修正アップコミットをリベースまたは追加する必要がありますか?

ソフトウェア開発における一般的なシナリオは、誰か他の人のコードをレビューするコードです。これを行うための一般的なツールは、プルリクエストを開くことです。

私の質問は、レビューで問題が見つかった場合、変更は

  1. 個別にコミットする(新しいコミット)
  2. または、既存のコミットを変更する必要があります(共有ブランチから履歴を書き直すのは悪いニュースなので、以前のコミットから誰もブランチしていないと仮定します...)。

最初のシナリオでは、コミット履歴に多少のノイズを追加しますが、増分変更を追跡するのは簡単です。 2番目のオプションには、逆の長所と短所があります。

11
loeschg

修正によって新しい問題が発生することはなく、古い問題は完全に修正されたと想定しています。しかし、多くの修正は、それ自体でレビューする価値があります。増分変更を個別にレビューできる場合は、おそらくはるかに簡単です。

23
Doc Brown