Att ge och ta emot code review utan att förstöra relationen

En av de mest kritiska processerna i modern mjukvaruutveckling är kodgranskning (Code Review, eller Pull Request/Merge Request). Det är här kodkvaliteten säkras, buggar fångas och kunskap sprids inom teamet. Men det är också här känslor kan bli sårade och arbetsrelationer prövas. I Sverige, där arbetskulturen präglas av konsensus och en stark vilja att undvika konflikter, är konsten att ge och ta emot kodgranskningar något av en delikat balansgång. Man vill upprätthålla en hög teknisk standard, utan att trampa någon på tårna.

Konstruktiv feedback: Fokusera på koden, inte personen

Den gyllene regeln vid kodgranskning är att alltid skilja på kod och kodare. En kommentar i en Pull Request (PR) ska aldrig uppfattas som en personlig attack. Istället för att skriva: 'Varför skrev du den här funktionen så här dåligt?', vilket låter anklagande, är det mycket bättre att använda ett inkluderande vi och ställa nyfikna frågor.
En svensk Tech Lead skulle snarare skriva: 'Intressant lösning! Vad tror du om att vi använder en map-funktion här istället för en for-loop? Det kanske gör koden lite mer lättläst för framtiden.' Denna konstruktiva ton öppnar upp för en dialog istället för att bygga en försvarsmur.

Akta dig för Nitpicking

Ett vanligt problem bland seniora utvecklare är så kallad nitpicking (att klaga på småsaker). Det handlar om att lämna dussintals kommentarer om exakt var ett kommatecken ska sitta, eller att en variabel borde ha ett marginellt bättre namn. I ett högpresterande svenskt team har man ofta löst detta genom automatiserade verktyg (Prettier, ESLint). Låt maskinerna bråka om formateringen! Om du ändå måste kommentera på en småsak, markera det tydligt: 'Nit: Det här är bara en liten detalj, men vi brukar döpa booleaner med is i början. Ignorera om du vill.' Det visar omtanke om kollegans tid.

Psykologisk trygghet i praktiken

Att ta emot en kodgranskning med femtio röda kommentarer kan få vem som helst att känna sig värdelös. Den svenska kulturen lägger stor vikt vid psykologisk trygghet. Det innebär att teamet ska kännas som en säker plats att göra misstag på. Om du som junior får mycket feedback, kom ihåg: det är inte ett underkännande av din kompetens. Seniora utvecklare granskar noggrant för att de bryr sig om produkten och för att de vill att du ska lära dig.

Ett bra svar när någon pekar ut ett uppenbart fel du gjort är att vara prestigelös. Du kan svara i PR:en: 'Bra fångat! Jag missade helt den detaljen. Fixar det direkt.' Genom att sänka garden och inte ta det personligt, bygger du enorm respekt hos dina kollegor.

Att godkänna (Approve) och fira små segrar

Kodgranskning handlar inte bara om att leta fel. Ett team med god kultur använder PR:s för att ge beröm. Att skriva 'Riktigt snyggt refaktorerat!' eller slänga in en passande emoji när någon har löst ett komplext problem, gör underverk för lagandan. När koden väl uppfyller er gemensamma standard, trycker man stolt på Godkänna (Approve).

Ta det offline vid stora konflikter

Ibland fastnar man. Du och din kollega kan ha fundamentalt olika åsikter om en arkitektur, och kommentarstråden på GitHub börjar bli alldeles för lång och passiv-aggressiv. Den svenska lösningen? Ta det offline. Skriv: 'Ska vi ta en snabb fika eller ett videosamtal och kika på detta tillsammans?' Fem minuters prat med empati och delad skärm löser ofta missförstånd som text aldrig kan reda ut. Relationen med ditt team är alltid viktigare än en enskild rad kod.