Testing copying from Google Docs

Vem gör Code Review?

Att utse en, eller ett par specifika personer för att utföra reviews skapar lätt flaskhalsar i organisationen som bör undvikas. Om vi är fler som utför reviews vinner vi både flexibilitet tidsmässigt och kunskapsspridning genom att fler får insikt i kod ur projekt de kanske normalt inte jobbar i. Personen som utför review bör:

Ha rätt kompetens – den som gör code review bör förstå koden som skrivits.
Först och främst vara en del av kund-teamet – då detta gynnar kunskapsspridning i samband med utvärdering.
Om inte teammedlem finns tillgänglig fråga i developers.

När gör vi code reviews?

Det finns många utvärderingar om hur code reviews görs på bäst sätt och de flesta är överens om att code reviews bör utföras ofta och med mindre antal kodrader vid varje tillfälle. Ju oftare vi har möjlighet att utföra dem, desto snabbare går de att genomföra, vilket i sin tur ger mindre tröskel för att utföra utvärderingen.

All kod utom väldigt små förändringar bör omfattas av code reviews i mitt tycke.

Det kan vara svårt att göra en review innan en feature är färdigutvecklad, men om vi kan bygga mer uppdelat i komponenter kanske detta skulle vara möjligt?

Diskussion:

Är kod för föränderlig för att utvärderas tills utveckling av en feature är ‘klar’?
Kan vi skapa bra rutiner för att bygga bra kod som kan utvärderas i delar?

Pull requests

Varje code review bör börja med en pull request i Github där utvecklaren som skrivit koden beskriver vad som har gjorts och om det finns något särskilt som behöver tas upp angående den berörda koden. Notera också om det finns något särskilt flöde av filer vid utvärderingen. Tänk på att den som utför code review kanske inte är insatt i koden och att kommentarer om olika delar kan guida och förenkla.

En pull request kan skapas innan en feature är klar, då märkt med WIP eller annat vi bestämmer.

Hur skapar vi en bra pull request?

Vi skriver en beskrivning av vad som förändrats eller lagts till i koden.
Vi noterar med kommentarer i koden där förändringar gjorts och guidar den som utvärderar koden.