Brown Bag: Slå varandra på fingrarna

4,574 views

Published on

Janko Luin har haft förmånen att jobba i ett team där kodgranskning var en naturlig, daglig och obligatorisk del av arbetsflödet. Det hade intressanta effekter, dels på kodens kvalitet och dels på gruppdynamiken.
Det bjuds frukost, dragning och diskussioner om vad som händer rent konkret när man börjar korrekturläsa varandras kod, och under vilka omständigheter det kan vara ett bra verktyg.
Utlovas: Ingen kod - bara principer och interaktioner!

0 Comments
0 Likes
Statistics
Notes
  • Be the first to comment

  • Be the first to like this

No Downloads
Views
Total views
4,574
On SlideShare
0
From Embeds
0
Number of Embeds
1,213
Actions
Shares
0
Downloads
2
Comments
0
Likes
0
Embeds 0
No embeds

No notes for slide

Brown Bag: Slå varandra på fingrarna

  1. 1. Slå varandra på fingrarna! Eller: Kodgranskning och vad det kan göra för ett team ! ! Janko Luin @gufo
  2. 2. "Vem f-n skrev den här skitkoden?"
  3. 3. "Vem skrev den här skitkoden?"
  4. 4. "Vem släppte igenom den här skitkoden?"
  5. 5. • Definitioner • Teamet • Observerade effekter • Gotchas • Tips på verktyg
  6. 6. “Code review is systematic examination (often known as peer review) of computer source code.” –Wikipedia
  7. 7. Informellt Parprogrammering Snabbkoll vid skärmen Pull Request Hyperformellt Fagan Inspection
  8. 8. Möt teamet • Kanban, TDD, BDD, CI, autodeploys, ... • GitHub + pull requests • Granskning som del av processen
 (kolumn på Kanban-tavlan)
  9. 9. Säg hej till detta: • Skriva för publik • Bli synad i sömmarna • Redaktörskap • Högre kodkvalitet
  10. 10. Säg hej till detta: • Oftare, mindre förändringar • Renare kodhistorik*
 * git only • Snabb assimilering av nya medlemmar
  11. 11. “Det var som att bli slagen på fingrarna” –Andreas
  12. 12. Glöm detta: • Din egen kodstil • Din egen kod • Personlig prestige
  13. 13. När vi slutade granska • Hög kodkvalitet
 (men lägre än innan) • Klantfel – "English version goes here" • Lite som att skriva kod utan tester...
  14. 14. Gotchas • Granskning tar tid...
 (men det tar tid att läsa kod) • Labbdag/städdag/refaktorering • Kräver verktygsstöd från dag 1
  15. 15. Är det för alla? – Ja och nej…
  16. 16. Verktyg

×