Code Reviews FTW!

V SUSE jsme v některých týmech cca před rokem začali používat code reviews. Jaké jsou naše zkušenosti a jaké vidíme výhody? Jak vlastně vypadá správné code review? Na co by se měl reviewer zaměřit a čemu se naopak vyhnout? A mají code reviews smysl i pro vás? Na tyto otázky zkusíme dát odpověď.

Předneseno na Pražské PyVo #25.

Video: http://youtu.be/0Gd16Pumo6c
Detaily: http://lanyrd.com/2013/praha-pyvo-april/scftwg/

Transcript

1. Praha, 17. dubna 2013 David Majda @dmajda Josef Reidinger @pepareidinger Code Reviews FTW!
2. Kontext 1
3. Proč? 2
4. lepší kód lepší design bus factor učení odhad kvality tlak okolí
5. Jak? 2
6. Úprava kódu Pull request Review Merge Přijetí Odmítnutí
7. Checklist 3
8. 1. Je kód dobře čitelný? 2. Rozumím tomu, co kód dělá? 3. Skautské pravidlo 4. Jde to udělat lépe?
9. Kvalita kódu 4
10. Viděl jsem to už někde?
11. Rozšiřitelnost
12. Pokrytí testy
13. Ošetření chyb
14. Štábní kultura
15. Správná úroveň abstrakce?
16. Izolace změn
17. Chyby se rády druží!
18. Overengineering!
19. Sociální aspekty 5
24. Problémy 6
25. Velké větve
26. Nekonečný příběh
27. Jednejte, jako kdybyste daný kód od zítřka měli na starost!
28. David Majda @dmajda Josef Reidinger @pepareidinger