RZECZOWNIK

code review

Czytanie 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. Najczęstsza uwaga dotyczy wcięcia, bo to jedyne, co widać bez czytania.

po ludzku: stempel

Co znaczy „code review”?

„code review” po ludzku znaczy: stempel.

W mailu

„code review to wąskie gardło”: po ludzku: wąskim gardłem jest zawsze cudze review
„dbamy o kulturę code review”: po ludzku: nikt nie czyta cudzego kodu
„compensation review”: po ludzku: przegląd bez skutków
„zróbmy code review”: po ludzku: przystemplujmy
„zrobić code review”: po ludzku: przystemplować

W drugą stronę: powiesz „rozmowa, po której wiesz, czy dostaniesz podwyżkę”, wyjdzie „performance review”.

Odmiana i pisownia

code review — także: review. Każda z tych pisowni znaczy to samo.

Podaj dalej

X LinkedIn Facebook Wykop Pobierz kartę PNG

Powiązane hasła

zrobić code reviewPrzegląd cudzego kodu przed scaleniem. Skuteczność spada gwałtownie wraz z rozmiarem zmiany — więc największe i najgroźniejsze zmiany dostają w praktyce najpobieżniejszy przegląd. 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. LGTMLooks good to me: wygląda dobrze. Formuła akceptacji w przeglądzie kodu, wypowiadana z dokładnością odwrotnie proporcjonalną do liczby zmienionych plików. sześćdziesiąt komentarzy w przeglądzie koduPrzeglą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… TL;DRToo long; didn’t read: za długie, nie przeczytałem. Postawione na początku własnego tekstu jest streszczeniem, postawione pod cudzym — recenzją. przegląd sprintuPodsumowanie iteracji. W praktyce w dużej części polega na wyborze tego, co nadaje się do pokazania.