Od prechodu na Git máme pull requesty jako povinnou součást workflow. Žádný kod se nedostane do develop bez review. Po šesti měsících sdílíme zkusenosti.
Proces¶
Feature branch, kod, push, pull request. Minimálně jeden reviewer approve. Komentáře, opravy, schválení, merge.
Checklist¶
Splňuje user story? Unit testy? Neduplikuje kod? Srozumitelne pojmenování? Security problemy? Error handling? Čitelnost?
Kultura¶
Review není kritika autora, je to vylepšení kodu. Konstruktivní tón. Ne toto je spatne, ale zvazoval bys toto řešení.
Vysledky¶
Prumerne 2.3 bugu na PR nalezeno pred merge. Knowledge sharing. Konzistentní kodova zakladna. Průměrná doba review: 30 minut.
Pravidla¶
- Review do 24 hodin. 2. Male PR (pod 200 radku). 3. Popisny PR description. 4. Konstruktivní tón.
code reviewkvalitagitprocesy
Potřebujete pomoc s implementací?
Naši experti vám pomohou s návrhem, implementací i provozem. Od architektury po produkci.
Kontaktujte nás
Potřebujete pomoc s implementací?
Domluvit schůzku