ZWROT

sześćdziesiąt komentarzy w przeglądzie kodu

Przegląd, który przestał dotyczyć kodu. Liczba komentarzy powyżej pewnego progu nie mierzy jakości zmiany, tylko stan relacji między dwiema osobami — i to widać po tym, że reszta zespołu przestaje się odzywać.

po ludzku: dwie osoby i jedna sprawa, która nie była o kodzie

Co znaczy „sześćdziesiąt komentarzy w przeglądzie kodu”?

„sześćdziesiąt komentarzy w przeglądzie kodu” po ludzku znaczy: dwie osoby i jedna sprawa, która nie była o kodzie.

Podaj dalej

X LinkedIn Facebook Wykop Pobierz kartę PNG

Powiązane hasła

dbamy o kulturę code reviewDeklaracja wzajemnego przeglądania kodu. Sprawdzalna jednym pytaniem: ile zgłoszeń przeszło w zeszłym tygodniu z komentarzem innym niż zatwierdzenie. kod ma zapachyOcena jakości kodu. Metafora zapachu opisuje coś, co się wyczuwa, a nie mierzy — więc jest to jedyny zarzut techniczny, którego nie da się odeprzeć testem. to jest wystarczająco dobrePróg, przy którym przestaje się poprawiać. Ustawia go ten, kto oddaje pracę, a sprawdza ten, kto ją czyta — i te dwa progi rozjeżdżają się dokładnie o tyle, ile komu zależało. ponadprzeciętne wynikiOcena pracy powyżej normy. Przeciętną wyznacza reszta zespołu, więc ponadprzeciętność jednej osoby jest opisem rozkładu pracy, nie jej jakości — i obie strony zwykle o tym wiedzą. code reviewCzytanie cudzego kodu przed scaleniem. Formalnie: wychwycenie błędów. Praktycznie: im większa zmiana, tym krótsza recenzja — przy tysiącu linii nikt nie przyzna, że nie doczytał, więc pisze LGTM. spotkanie o architekturzeDwie godziny na ustalenie kierunku, który i tak zostanie rozstrzygnięty w kodzie. Wartość takiego spotkania jest odwrotnie proporcjonalna do liczby uczestników, którzy tego kodu nie napiszą.