マージ前にプルリクエストをクリーニングするにはどうすればよいですか？

Question

GitHubのプルリクエストメカニズムに関する質問があります。

ボブはその枝でいくつかの変更をコミットします：

* Added feature foo 
* Added feature bar 
* Removed latency in baz 

は、それから彼はPRを作成します。アリスと他の人々はPRを拒否し、変更を求める。ボブは、彼らが満足していないとボブは終わり

* Again some change, please accept my PR 
* Damned, I forgot this... 

怒るので、彼らは私たちは誰もがと一致するものを持って、より多くの変更を求めるが、私たちは騒々しいコミットをしてい

* !fixup Added feature foo 
* !fixup Added feature bar 
* cosmetic change 

再び新しいコミットを追加します。だから、3つのオプションが残っている：

1. 我々はPRを受け入れるが、それはすべてのそれらのコミット
2. ボブrebase -iとpush -f --force-with-lease（？しかし、何のPRに起こる）と醜くなる
3. ボブは上のPR、rebase -iを閉じ新しい支店が新しいPRを作成します。

最適なオプションは何ですか？

P.S. squashは、のプルリクエストを行い、最初の3つのコミットを維持したいので、オプションではありません。foo、bar、遅延問題を修正しました。

Answer 1

第2のオプションは正常です。プル要求は、rebase -iオペレーションの新しいコミットで更新され、以前のコミットが置き換えられます。これらの新しいコミットはプルリクエストの最後のコメントの後に表示され、レビュー担当者はレビューする新しいコミットがあることを知ることができます（少なくとも以前と同じ内容であることを確認する）。