Download - Brown Bag: Slå varandra på fingrarna
Transcript
Slå varandra på fingrarna!Eller: Kodgranskning och vad det kan göra för ett team
!
!
Janko Luin @gufo
–Wikipedia
“Code review is systematic examination (often known as peer review) of computer source code.”
Möt teamet
• Kanban, TDD, BDD, CI, autodeploys, ...
• GitHub + pull requests
• Granskning som del av processen (kolumn på Kanban-tavlan)
Säg hej till detta:
• Oftare, mindre förändringar
• Renare kodhistorik* * git only
• Snabb assimilering av nya medlemmar
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...
Gotchas
• Granskning tar tid...(men det tar tid att läsa kod)
• Labbdag/städdag/refaktorering
• Kräver verktygsstöd från dag 1
Verktyg