Od prechodu na Git mame pull requesty jako povinnou soucast workflow. Zadny kod se nedostane do develop bez review. Po sesti mesicich sdilime zkusenosti.
Proces¶
Feature branch, kod, push, pull request. Minimalne jeden reviewer approve. Komentare, opravy, schvaleni, merge.
Checklist¶
Splnuje user story? Unit testy? Neduplikuje kod? Srozumitelne pojmenovani? Security problemy? Error handling? Citelnost?
Kultura¶
Review neni kritika autora, je to vylepseni kodu. Konstruktivni ton. Ne toto je spatne, ale zvazoval bys toto reseni.
Vysledky¶
Prumerne 2.3 bugu na PR nalezeno pred merge. Knowledge sharing. Konzistentni kodova zakladna. Prumerna doba review: 30 minut.
Pravidla¶
- Review do 24 hodin. 2. Male PR (pod 200 radku). 3. Popisny PR description. 4. Konstruktivni ton.
code reviewkvalitagitprocesy