0
プルリクエストを確認していて、コードを改善できる方法があるとします。開発者が元のコミットを修正するか、変更を加えて新しいコミットを作成し、変更の理由を説明するコミットメッセージを作成することをお勧めしますか？エラーを修正する新しいコミットを追加するのではなく、機能のリリースでコミットを修正する必要がありますか？
A
答えて
2
個人的には、私は新しいコミットの作成をお勧めします。
このようにして、後で改善が逆になる場合に備えて、古いコードが引き続き追跡可能になります。
すべてのコミットで乱雑になったら、いつでもrebaseすることができます:)
関連する問題
- 1. Fossilでコミットを修正する
- 2. 不正なパターンエラーを修正する必要があります
- 3. このエラーを修正する必要があります
- 4. コミット後のSVNコミットが非常に遅くなります - 修正方法？
- 5. ヘッダー "fcntl.h"でこのエラーを修正する必要はありますか？
- 6. 修正バージョン1コミット
- 7. 新しい変更をコミットする前に直前のコミットを修正してください
- 8. Endeca CASのデータを修正する必要があります
- 9. java tcpバグを修正する必要があります
- 10. dingo apiバリデータールールを修正する必要があります
- 11. スクラップPHPスクリプトを修正する必要があります
- 12. subversionでコミットされた間違いを修正する
- 13. 既にコミット済みのCRLFをgitで修正します
- 14. 最後のgitコミットの修正を
- 15. スクリプトで2つの警告を修正するにはどうすればよいですか？警告は修正する必要がありますか？
- 16. サブバージョンを使用して修正されたファイルをコミットする
- 17. git rebaseで壊れているGitHubプルリクエストのコミット順を修正するには？
- 18. アンドロイドのエラーを修正する方法 '色の値は＃で始める必要がありますか？'
- 19. 基数ソートはconst値で修正する必要がありますか？
- 20. マージの前にGitコミットのメッセージを修正する
- 21. 多くのコミットが追加された後にコミットをコミットする方法はありますか？
- 22. Strutsをアップグレードする際に修正する必要はありますか？
- 23. Excel VBAエラー9サブアウトオブレンジ - 最終行/コラムを修正する必要がある可能性があります
- 24. ナビゲーションCSSの問題を修正する必要がありますか？
- 25. Mercurial：前のコミット日付を修正する
- 26. プロジェクトで.rvmrcファイルをコミットする必要がありますか？
- 27. Chart.jsはx軸の数値を修正する必要があります
- 28. コミットする前に、追跡対象のファイルを追加する必要があるのはなぜですか？
- 29. いくつかの修正が必要ですSurfaceAreaとLateralArea
- 30. ChunkedEncodingErrorはどのように修正する必要がありますか？
これは本当に意見の問題です。いくつかの人は、それぞれのPRには1つのコミットがあり、いくつかはいくつかのコミットを好むことを好む – pedrorijo91