병합하기 전에 끌어 오기 요청을 정리하는 방법은 무엇입니까?

Question

GitHub의 풀 요청 메커니즘과 관련된 질문이 있습니다.

밥은 지점에 일부 변경을 커밋 :

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

그런 다음 그는 PR을 만듭니다. 앨리스와 다른 사람들은 홍보를 거부하고 변화를 요구합니다. 밥은 행복하지와 밥이 말

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

화가 얻을 수 있도록 그들은 우리가 모두가 동의 뭔가를 더 많은 변화를 요구하지만, 우리는 시끄러운 커밋이

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

다시 새로운 커밋을 추가 . 그래서 세 가지 옵션이 남아 :

1. 우리는 PR을 허용하지만, 모든 커밋과 추한 도착
2. 밥 rebase -i 및 push -f --force-with-lease (하지만 홍보는 어떻게됩니까?)
3. 밥이의 PR, rebase -i을 닫습니다 그런 다음 새로운 지점이 새로운 PR을 만듭니다.

어떤 옵션이 가장 좋습니까?

P. 은 풀 요청을 rebase에 넣고 foo, bar 및 대기 시간 문제를 해결하는 초기 세 가지 커밋을 유지하기 때문에 옵션이 아닙니다.

Answer 1

두 번째 옵션이 정상적으로 보입니다. 끌어 오기 요청은 이전 커밋을 대체하여 rebase -i 작업의 새 커밋으로 업데이트됩니다. 이 새로운 커밋은 끌어 오기 요청에 대한 마지막 주석 뒤에 표시되어 검토자가 검토 할 커밋이 있다는 것을 알 수 있습니다 (적어도 이전과 같은 내용인지 확인).