Kod do danego taska zazwyczaj tworzy jedna osoba ale całość dowozimy jako zespół. Code review jest integralną częścią procesu i nie powinniśmy zostawiać tego na ostatnią chwilę.
Nie skupiajmy się na detalach jak coś zostało zaimplementowane a na szerszej perspektywie. Nie traćmy czasu na dyskusje o preferencjach syntaxu A nad syntaxem B
const fun1 = function () {
...
}
function fun2() {
...
}
const fun3 = () => {
...
}Sporne kwestie rozwiążmy na początku i spiszmy w postaci kontraktu.
Nie bójmy się korzystać z długu technicznego.
Trzymajmy dyscypline w śledzeniu go i stopniowym rozwiązywaniu.
Self-review przed review od zespołu
Code review request kiedy PR jest w 100% gotowy
Przetestowanie lokalne ficzera - nie tylko review na github.
Ułatwmy review dla zespołu na tyle na ile to możliwe
Dodanie komentarzy w PR w miejscach o których wiemy że mogą wzbudzić wątpliwości
Opisanie drogi która doprowadziła nas do danego rozwiązania
Dodanie screenshotów
Podlinkowanie w opisie wszystkich powiązanych źródeł