プルリクエストで要求された変更を含める正しい方法

Question

要求された変更をPRに含める正しい方法は何でしょうか？

査読者が自分のPRに何らかの変更をリクエストしたとします。メソッド名の変更、依存関係の取り消しの取り消しなど

私は変更を加えて新しいコミットをしますか？
またはdo I change past commit to include the requested changes？

or reset and change

Answer 1

私はそれが強く、あなたのチーム編成に依存していることだと思います。

たとえば私のチームでは、自分のPRで要求が変更された場合は、投稿PRのコミットでグループ化し、そのコメントをRework after PR: <my changes here>で開始しようとします。

ここでいくつかの良いリソース：
- Understanding Github flow
- 一般git-flow cli

、私は（PR後に）変更内容をcommit --amendしないことをお勧めします、それは物事のトラックを失うための最良の方法だあなたは、後で必要になる場合があります。

Answer 2

いいえ、最後のコミットを変更する必要はありません。通常の処理、つまりファイルの変更は、git commit -am "Bug_id"を実行してからgit pushを実行するだけです。査読者のコメントの後に自動的にの下にリストされます。

Answer 3

他の人が言っているように、主に保守者が従う/他人に採用するよう頼む慣行に大きく依存しています。

私は通常、査読者が行った変更を、前のものを修正するのではなく別のコミットにコミットします。これにより、査​​読者が変更を追跡しやすくなります。あなたがPRに満足しているとき、彼らは通常、gitの歴史を見ている人にとってより意味をなさないように、あなたにコミットを縮退/修正するよう依頼します。そのためにgitのすばらしいインタラクティブなリベース機能を使用してください：https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History。これは最初は威圧しているかもしれませんが、あなたが理解しマスターしたら、非常に多彩なツールです。

ただし、変更が少なく、重要でない場合や、別のコミットが必要ない場合は、変更を加えて直前のコミットを修正する方がよいでしょう。 gitは最後にコミットしたコミットの変更が子孫（つまり、特定のコミットの上にコミットされた変更）であることをデフォルトで要求するので、git push -fを使用することを忘れないでください。