( denesb | 2020. 05. 27., sze – 12:58 )

Semmi különleges codereview-t nem csinálunk. Minden patch-nek megvizsgáljuk a motivációját (tényleg kell ez a változás), a designját, teljesítményre való hatását, helyességet, olvashatóságot, és -- messze a legfontosabb --, hogy helyesen van-e használva a whitespace. Szerintem ez a standard codereview. A GH annyi a baj, hogy egyfelől nagyon le van butítva: pl a hozzászólások nincsenek szálakba rendezve (mint pl itt a HUP-on), szóval ha nem csak 1-2 válasz van valamire akkor hamar olvashatatlan lesz. Másfelől csak egyféle workflow-al működik jól: ha nem érdekel a git history-d és minden review commentet külön commitokba orvosolsz. Ha force push-olod az új verziót (ahogy mi csináljuk) akkor hamar szétesik az egész beszélgetés folyamat.

És persze van egy csomó idegesítő apróság: a commitok dátum szerint vannak rendezve, nem természetes sorrendbe. Az emal értesitések teljesen hasznavehetetlenek, mivel nincs szövegkörnyezet bennük, illetve nem lehet emailbe válaszolni rájuk (az emailben küldött választ a PR-hez csatolja, nem a hozzászóláshoz amire válaszolsz).