풀 요청을 수정하기위한 해결책은 무엇입니까?

Question

커밋이 거의없는 GitHub (Enterprise)에서 홍보를했습니다. 내 검토가 하나의 작은 실수를 발견 커밋 :

* 0c99cf29 Reapply Did this (HEAD -> foo) 
* 8806f36b Reapply Did that 
* 572e1122 Reapply Done that 
* 64ea3dc8 Reapply Accomplished this 
* 81e20976 Fix this (this time correctly) 
* d78a4534 Revert Fix this <-- Error there 
* c0d817a9 Revert Accomplished this 
* ed2bb3b2 Revert Done that 
* ea34322a Revert Did that 
* f81b78a3 Revert Did this 
* a401341c Did this 
* 08e97f86 Did that 
* 616cd4ad Done that 
* f3c6151b Accomplished this 
* 1af6e74f Fix this <-- Error there 
* a099fc19 Finished this 
* ab726eb3 Cherry-picked this (master, origin/master) 

: 그냥 다음, 충돌없이 되돌아 보정 모든 커밋을 다시 적용 할 수 없기 때문에

는

* a401341c Did this (HEAD -> foo) 
* 08e97f86 Did that 
* 616cd4ad Done that 
* f3c6151b Accomplished this 
* 1af6e74f Fix this <-- Error there 
* a099fc19 Finished this 
* ab726eb3 Cherry-picked this (master, origin/master) 

첫 번째 솔루션은 1af6e74f 후 모든 커밋을 되돌리려하는 것입니다

다음 내 PR을 업데이트하려면 git push.

두 번째 솔루션은 git push -f

git checkout 1af6e74f 
git commit --amend -am "Fix this (with corrections)" 
git rebase --onto a401341c f3c6151b HEAD # Not sure this will work as written 
git branch -f foo HEAD 
git push -f 

은 이전 솔루션 좋은 해결책과 나쁜 항상 후자가 포함까요?

Answer 1

질문 :

기존 지점과 새로운 지점 (자식은 diff a401341c의 0c99cf29) 사이의 DIFF는 무엇입니까?
버그가 수정 된 방법을 명확하게 나타내는 합리적인 패치처럼 보입니까? 만약 그렇다면

---

, 당신의 이전 분기의 상단에 커밋 새로운 내용을 새로운으로이 커밋 :

git checkout foo 

# just to be on the safe side : work on a new temporary branch 
git checkout -b wip 

# go back to the old sate : 
git reset --hard a401341c # <- original "Did this" commit 

# get the content from new branch : 
git checkout 0c99cf29 . # <- don't forget the "." 

# check that it matches what you expect : 
git diff [--cached] ... 
git difftool -d [--cached] ... 

# if OK : commit ! 
git commit 


# make your local "foo" branch point to this commit : 
git checkout foo 
git reset --hard wip 
git branch -d wip 

# push : 
git push origin foo