Slå varandra på fingrarna!Eller: Kodgranskning och vad det kan göra för ett team

!

!

Janko Luin @gufo

"Vem f-n skrev den här skitkoden?"

"Vem skrev den här skitkoden?"

"Vem släppte igenom den här skitkoden?"

• Definitioner

• Teamet

• Observerade effekter

• Gotchas

• Tips på verktyg

–Wikipedia

“Code review is systematic examination (often known as peer review) of computer source code.”

Informellt

Fagan Inspection

Pull Request

Snabbkoll vid skärmen

Parprogrammering

Hyperformellt

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:

• Skriva för publik

• Bli synad i sömmarna

• Redaktörskap

• Högre kodkvalitet

Säg hej till detta:

• Oftare, mindre förändringar

• Renare kodhistorik* * git only

• Snabb assimilering av nya medlemmar

–Andreas

“Det var som att bli slagen på fingrarna”

Glöm detta:

• Din egen kodstil

• Din egen kod

• Personlig prestige

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

Är det för alla?– Ja och nej…

Verktyg