Ez egy olyan PR, amihez négy kommit volt.
Igen, és már itt látszik a baj: a negyedik commit egy olyan hibát (missing EOF) javít, amit a harmadik commit vezetett be. Ilyenkor nem a tetejére dobunk rá egy fixupot, hanem megjavítjuk a harmadik patch-et, és az egész branch-et újra feltoljuk. (Az a tény, hogy itt ez az eljárás eleve a v1-en belül történt (vagyis hogy a "tetejére dobott fixup" már a legelső push-nak a része volt), számomra még inkább elfogadhatatlan. Ki a csuda tol fel úgy egy patch series-t, hogy saját maga is tudja, hogy egyes patch-ek a sorozatban félkészek, hibásak?)
Changes from all commits
Igen, ez a kumulatív diff; áttekinteni hasznos lehet, kommentáláshoz nagyrészt haszontalan, mert nem követi azt a vonulatot, evolúciót, ahogyan a végállapot kialakul.
Show changes since your last review
Ilyet még nem láttam, és (mivel ezen a PR-en nincs több, külön push / review) itt nem is tudom megnézni.
commitonként is megnézheted a változásokat
A fő kérdés az, hogy a legutolsó review óta és commit-onként láthatom-e a változásokat. Igazából engem nem a git diff érdekel, mert nem (csak) a kód változása érdekel. Engem az egyes patch-ek változása érdekel (diff-ek diff-je --> interdiff).