Kodgranskning med en spansk kollega
Ge och ta emot feedback på kod på ett artigt sätt med en spansktalande utvecklare.
I spanska techteam sker kodgranskningen i PR-kommentarer och snabba samtal, med massor av engelska inblandat: la PR, los tests, mergear. Feedback mjukas upp med konditionalis och små garderingar som ”una tontería” eller ”es solo una sugerencia” – ungefär som när en svensk säger ”bara en liten grej”. Korten följer två utvecklare som granskar varandras pull requests.
- 1
Una tontería: quizás cambiaría el nombre de esta variable.
En petitess: jag kanske skulle döpa om den här variabeln.
- 2
Es solo una sugerencia, no es bloqueante.
Det är bara ett förslag, det är inte blockerande.
- 3
No acabo de entender qué hace esta función.
Jag förstår inte riktigt vad den här funktionen gör.
- 4
¿Podrías añadir un comentario aquí?
Skulle du kunna lägga till en kommentar här?
- 5
Esto se podría simplificar.
Det här skulle kunna förenklas.
- 6
Hay mucho código duplicado.
Det är mycket duplicerad kod.
- 7
¿Has tenido en cuenta el caso de que la lista esté vacía?
Har du tänkt på fallet där listan är tom?
- 8
Bien visto, no lo había pensado.
Bra fångat, det hade jag inte tänkt på.
- 9
Ahora lo arreglo.
Jag fixar det nu.
- 10
Lo he hecho así a propósito.
Jag gjorde så med flit.
- 11
Te explico por qué.
Jag ska förklara varför.
- 12
Vale, ahora tiene más sentido.
Okej, nu är det mer begripligt.
- 13
Están fallando los tests en el CI.
Testerna fallerar i CI:n.
- 14
En mi máquina funciona.
Det funkar på min dator.
- 15
Preferiría dividir esto en dos PR.
Jag skulle hellre dela upp det här i två PR:er.
- 16
¿Le puedes echar otro vistazo cuando puedas?
Kan du kika på den igen när du får tid?
- 17
Te he dejado unos cuantos comentarios.
Jag har lämnat några kommentarer till dig.
- 18
He aplicado los cambios que me sugeriste.
Jag har gjort ändringarna du föreslog.
- 19
Por mí está bien, te la apruebo.
Ser bra ut för min del, jag godkänner den.
- 20
Ya la puedes mergear.
Du kan merga den nu.