Co idzie dobrze?

Co sprawdza się w naszych przeglądach kodu?

Checklista do code review pomogła mi skupić się na najważniejszych obszarach.
Doceniam szczegółowy feedback, naprawdę pomógł mi lepiej zrozumieć kod.
Przeglądanie kodu różnych osób z zespołu pokazało mi nowe techniki.
Co nie idzie dobrze?

Z jakimi trudnościami mierzyliśmy się podczas przeglądów kodu?

Przeglądy często się opóźniają przez zmieniające się priorytety i terminy.
Komentarzom czasem brakuje kontekstu, przez co trudno je zrozumieć.
Zbyt krytyczny feedback potrafi zdemotywować ludzi w zespole.
Co możemy poprawić?

Jak możemy usprawnić nasz proces i praktyki code review?

Wprowadźmy lekkie narzędzie, które usprawni przepływ pracy przy przeglądach.
Ustalmy zasady dotyczące zakresu przeglądu i dopuszczalnej wielkości zmian.
Zorganizujmy szkolenie ze skutecznego przeglądania kodu i dobrych praktyk.
Jakie pytania się pojawiły?

Jakie nowe pytania lub obszary do zbadania się pojawiły?

Jak lepiej wykorzystać narzędzia do analizy kodu w naszych przeglądach?
Jakie są dobre praktyki branżowe przy przeglądaniu kodu legacy?
Jak skuteczniej godzić jakość kodu z terminami dostarczenia?

Czym jest retrospektywa code review?

Retrospektywa code review to spotkanie, na którym zespół przygląda się temu, jak przegląda kod, a nie samemu kodowi. Pull requesty czekające całymi dniami, komentarze bez kontekstu, przeglądy sprawdzające wyłącznie styl: takie problemy rzadko wychodzą na zwykłej retrospektywie sprintu, bo są rozłożone na cały tydzień pracy wszystkich osób, a nie związane z jednym zdarzeniem. Tablica ma cztery kolumny. „Co idzie dobrze?” zbiera nawyki warte zachowania, takie jak przydatna checklista czy życzliwy ton. „Co nie idzie dobrze?” obejmuje bolączki, od zbyt dużych pull requestów po niespójne standardy. „Co możemy poprawić?” zamienia je w konkretne zmiany, na przykład limit wielkości zmian albo uzgodniony czas odpowiedzi na przegląd. „Jakie pytania się pojawiły?” to miejsce na otwarte pytania, które zespół musi zbadać, zanim podejmie decyzję. Przeprowadź ją po dużym wydaniu, gdy opóźnienia w przeglądach zaczynają spowalniać dostarczanie, albo gdy dołączają nowe osoby i niepisane standardy przeglądów zespołu trzeba wreszcie spisać. Nie szukajcie winnych: tematem jest proces, a nie czyjeś komentarze do konkretnego pull requesta.

Format retrospektywy code review

Co idzie dobrze?

Co sprawdza się w naszych przeglądach kodu?

Zacznij od tego, co działa. Poproś o konkretne praktyki przeglądowe, które pomogły, i doceń osoby, których przeglądy coś zmieniły.

Co nie idzie dobrze?

Z jakimi trudnościami mierzyliśmy się podczas przeglądów kodu?

Zadbaj o to, by można było mówić o problemach bez obwiniania kogokolwiek. Poproś o konkretne przykłady, np. przegląd, który trwał kilka dni, albo komentarz, z którym trudno było cokolwiek zrobić.

Co możemy poprawić?

Jak możemy usprawnić nasz proces i praktyki code review?

Poproś o konkretne sugestie, które da się wdrożyć. „Lepsze przeglądy” to nie działanie; „dzielimy pull requesty, które dotykają więcej niż jednej funkcjonalności” już tak.

Jakie pytania się pojawiły?

Jakie nowe pytania lub obszary do zbadania się pojawiły?

Zapisz pytania, na które zespół nie potrafi dziś odpowiedzieć, i ustalcie, kto zajmie się każdym z nich przed następną retrospektywą.

Kiedy należy skorzystać z niniejszego przeglądu retrospektywnego

  • Gdy przeglądy spowalniają dostarczanie lub wywołują tarcia i chcesz dowiedzieć się dlaczego.
  • Gdy chcesz, żeby programiści podzielili się praktykami przeglądowymi, które się u nich sprawdzają.
  • Po dużych wydaniach lub projektach, żeby sprawdzić, jak przeglądy wytrzymały presję.
  • Gdy dołączają nowe osoby i trzeba uzgodnić standardy przeglądów, oczekiwania oraz czas odpowiedzi.

Proponowane pytania lodołamacze

  • Jaki najciekawszy kod przeglądałeś ostatnio?
  • Gdybyś mógł dodać jedną funkcję do swojego edytora kodu, co by to było?

Czy chciałby Pan/Pani, aby Państwa zespół wykonał ćwiczenie rozgrzewkowe, a nie tylko odpowiadał na pytania na głos? Przejrzyj bezpłatne lodołamacze →

Pomysły i wskazówki dotyczące spotkania retrospektywnego

  • Prowadź rozmowę bez szukania winnych, żeby ludzie szczerze mówili o przeglądach, które były dla nich trudne.
  • Doceniaj dobre przeglądy, np. jasne komentarze czy szybką odpowiedź, a nie tylko omawiaj problemy.
  • Zaproś wszystkich, którzy piszą lub przeglądają kod, w tym testerów.
  • Uzgodnijcie jedną lub dwie konkretne zmiany zamiast długiej listy dobrych intencji.
  • Wracaj do zadań do wykonania i śledź postępy na kolejnych retrospektywach.
  • Zmieniajcie osobę prowadzącą, żeby dyskusję prowadziły różne perspektywy.

Najczęściej zadawane pytania

Czym jest retrospektywa code review?
To spotkanie, na którym zespół zastanawia się, jak przegląda kod: co idzie dobrze, co nie, co poprawić i jakie pytania pozostają otwarte. Celem są przeglądy szybsze i bardziej przydatne dla osoby, która je otrzymuje.
Kiedy przeprowadzić retrospektywę code review?
Po dużym wydaniu lub projekcie, gdy opóźnienia w przeglądach zaczynają spowalniać dostarczanie, albo regularnie, żeby uzgodnić standardy przeglądów, zanim utrwalą się nawyki.
Jakie pytania zadać podczas retrospektywy code review?
Zacznij od czterech pytań z tablicy: co idzie dobrze, co nie idzie dobrze, co możemy poprawić i jakie pytania się pojawiły. Przydatne pytania uzupełniające to m.in. jak długo pull requesty czekają na pierwszy przegląd i czy komentarze wyjaśniają, dlaczego zmiana jest potrzebna.
Ile trwa retrospektywa code review?
Zaplanuj 45–60 minut. Jeśli zespół ma wiele problemów z przeglądami do omówienia, skupcie się na dwóch lub trzech, które zebrały najwięcej głosów.
Czym różni się od retrospektywy sprintu?
Retrospektywa sprintu obejmuje wszystko, co wydarzyło się w sprincie. Retrospektywa code review dogłębnie analizuje jedną praktykę, dzięki czemu problemy, które w pojedynczym sprincie wydają się drobne, jak powolne przeglądy, dostają wystarczająco dużo uwagi, by je naprawić.
Czy są Państwo gotowi do przeprowadzenia tej retrospektywy?Zapraszamy do obejrzenia tej retrospektywy na żywo