My talk about how to make code review
The other year I held a talk about how to make code reviews that are of value.
As zip.
Förvänta dig inte att din Pull request går igenom
ola fjelddahl
göteborgskontoret
Förvänta dig inte att din Pull request går igenom
PRAISE: Nice refactoring!
ola fjelddahl
göteborgskontoret
Förvänta dig inte att din Pull request går igenom
PRAISE: Nice refactoring!
SUGGESTION: Split line.
ola fjelddahl
göteborgskontoret
Förvänta dig inte att din Pull request går igenom
PRAISE: Nice refactoring
SUGGESTION: Isn’t it an Article?, not a Device.
CHECK: Have you investigated all uses of the enum?
ola fjelddahl
göteborgskontoret
Förvänta dig inte att din Pull request går igenom
PRAISE: Nice refactoring!
SUGGESTION: Split line.
CHECK: Do you use the enum the correct way? Are all cases considered?
EXPLAIN: How do this work?
ola fjelddahl
göteborgskontoret
Förvänta dig inte att din Pull request går igenom
PRAISE: Nice refactoring!
SUGGESTION: Split line.
CHECK: Do you use the enum the correct way? Are all cases considered?
EXPLAIN: How do this work?
FIX: Check for null.
ola fjelddahl
göteborgskontoret
Förvänta dig inte att din Pull request går igenom
PRAISE: Nice refactoring!
SUGGESTION: Split line.
CHECK: Do you use the enum the correct way? Are all cases considered?
EXPLAIN: How do this work?
FIX: Check for null.
QUESTION: Is this a HasA or a IsA
ola fjelddahl
göteborgskontoret