Jsme 4 vývojáři a zavádíme povinné code review. Zatím to ale drhne: PR visí dny, komentáře se točí kolem formátování a lidi to otravuje. Máte osvědčené postupy, jak review udržet rychlé a užitečné? Nějaká pravidla velikosti PR, SLA na reakci, automatizace?
1× díky
Formátování vyřeš automatem (linter/formatter v CI), ať se lidi baví o logice. Malé PR do 300 řádků a review do 24 hodin jako týmová dohoda.
Osvědčilo se nám párové programování u větších věcí – review je pak formalita.
4× díky
Linter v CI byl první krok, hádky o čárky zmizely. :)
5× díky