Git - co zrobić z review

Question

Zrobiłem forka pewnej biblioteki na GitHubie, troszkę pododawałem i poprosiłem o pull request który doczekał się review. Sugerowane zmiany powinienem dodać jako kolejny commit i poprosić o review?

Answer 1

Jak chcesz - możesz albo zrobić amend/rebase na istniejących commitach, albo dodać nowy commit z poprawkami. Jedni wolą jedno, inni drugie. Na koniec zrób (force, jeśli potrzebne) pusha na brancha z review i GH sam powinien zaktualizować stan.

Answer 2

są różne podejścia, ja raczej preferuję zrobienie nowego commita gdyż łatwiej wtedy wyśledzić zmiany od ostatniej wersji, rozwiązanie z amend raczej zostawiam na bardzo małe zmiany lub fixy, które sam zauważę zanim zanim jeszcze wypchnę zmiany albo otworzę PR.

Comments

Zmiany były dość obszerne, udało mi się stworzyć cuda typu (nie wiem jak to się stało) 

map[name] = value == 'true' ? true : false;

Dodałem jako commit i poprosiłem o kolejne review. Dzięki za odpowiedź 

---

Tomek, ale też z drugiej strony: wyobraź sobie że masz spory PR na 5 commitów i 2-3 osoby będą robiły review. Pierwsza już zrobiła (albo np automatyczne narzędzie zrobiło), zostawiła parę uwag. Jeśli w tym momencie zrobisz nowy commit, kolejny reviewer czytając zmiany po kolei, wciąż będzie natykał się na oryginalny błędny kod, aż nie dojdzie do nowych commitów z poprawkami. Z rebasem, kolejny reviewer będzie od razu czytał wstępnie poprawiony kod i nie będzie marnować czasu na uwagi, które już inne zgłosili.

łatwiej wtedy wyśledzić zmiany od ostatniej wersji

Niektóre narzędzia, jak reviewboard, potrafią śledzić "historię review", uwzględniającą amendy tego samego commita. Github niestety nie, z tego co wiem.