70. Code Review w zespole — co sprawdzać, jak dawać feedback i checklist

This content is not available in your language yet.

O czym jest to opracowanie?

Code Review w zespole

Code review to proces, w którym inni członkowie zespołu przeglądają kod przed jego włączeniem do głównej gałęzi (merge). Cel to nie kontrola, ale współpraca: wykrywanie błędów, dzielenie się wiedzą i utrzymanie spójności kodu. To standard w każdej profesjonalnej firmie IT.

Dlaczego warto to znać?

Code review jest obowiązkowym krokiem przed merge w GitHub/GitLab
Statystyki: bug znaleziony w review kosztuje 10x mniej niż na produkcji
Review uczy — zarówno autor kodu jak i reviewer uczą się wzajemnie

Cel Twojego opracowania

Wyjaśnić czym jest code review i jakie są jego cele
Opisać co sprawdzać podczas przeglądu kodu PHP
Omówić jak dawać konstruktywny, nieagresywny feedback
Przygotować checklistę reviewera

Zakres obowiązkowy

Definicja code review — nie ocena pracownika, a współpraca
Miejsce w procesie: PR/MR → review → approve → merge
Co sprawdzać: poprawność logiki, bezpieczeństwo, czytelność, testy
Jak komentować: pytania zamiast nakazów, propozycje nie krytyka
Conventional Comments — prefiksy: nit:, suggestion:, question:
Jak jako autor przygotować PR: opis, małe zmiany, testy
Checklist reviewera — lista punktów kontrolnych

Wymagane przykłady kodu

Przykład 1

Fragmenty kodu PHP z potencjalnymi problemami do review

Przykład 2

Przykładowe komentarze review: zły i dobry feedback

Przykład 3

Checklist reviewera PHP (min. 8 punktów)

Proponowana struktura pracy

Struktura minimalna

Wstęp — znaczenie współpracy w zespole
Czym jest code review
Co sprawdzać w code review PHP
Jak dawać konstruktywny feedback
Conventional Comments
Jak przygotować PR jako autor
Checklist reviewera
Podsumowanie i bibliografia

Pytania kontrolne

Najczęstsze błędy

Błąd	Problem	Rozwiązanie
Brak checklisty	Niekompletne opracowanie	Przygotuj listę min. 8 punktów
Pominięcie przykładu feedbacku	Teoria bez dowodów	Pokaż dobry i zły komentarz review
Opis ogólny bez kontekstu PHP	Mało wartości	Skupi się na code review kodu PHP

Zasoby

Google Engineering Practices Przewodnik Google po code review

Conventional Comments Standard formatowania komentarzy

GitHub: About Pull Requests GitHub PR documentation

Do dzieła!

Zrób review kodu kolegi!

Poproś kolegę o kod z projektu i przeprowadź review. Zapisz swoje komentarze. Opisz ten proces w opracowaniu — to najlepszy materiał z własnego doświadczenia.