Code Reviews FTW!

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 Devel.cz Konference 2013.

Detaily: http://devel.cz/konference/2013

Ebe96461709771a430da9c7c58f9ae5f?s=128

David Majda

March 02, 2013
Tweet

Transcript

  1. Praha, 2. března 2013 David Majda @dmajda Josef Reidinger (twitterless)

    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

  20. None
  21. None
  22. None
  23. None
  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. http://www.doporucim.cz/mobile/DEVELREVIEWS