Kodgranskning med en fransk kollega
Ge och ta emot kommentarer i kodgranskning på franska: mildra en petitess, fråga varför, säg emot artigt och godkänn mergen.
Franska utvecklare säger tu till varandra och gör kodgranskning på en blandning av franska och engelsk jargong: une PR, merger, un commit, les tests passent. Kommentarer mildras ofta med konditionalis (je renommerais, ’jag skulle döpa om’) eller med ’petit détail’ för en petitess, och ’c'est pas bloquant’ betyder att man kan merga ändå. Skrivna kommentarer är lite mer försiktiga än det man säger, men tonen är rak och vänlig.
- 1
Petit détail : je renommerais bien cette variable.
En liten petitess: jag skulle döpa om den här variabeln.
- 2
C'est juste une suggestion, c'est pas bloquant.
Det är bara ett förslag, det blockerar inte.
- 3
Je comprends pas trop ce que fait cette fonction.
Jag förstår inte riktigt vad den här funktionen gör.
- 4
Tu pourrais ajouter un commentaire ici ?
Skulle du kunna lägga till en kommentar här?
- 5
Ça pourrait être simplifié.
Det här skulle kunna förenklas.
- 6
Y a pas mal de code dupliqué.
Det är ganska mycket duplicerad kod.
- 7
Pourquoi t'es parti sur cette approche ?
Varför valde du den här lösningen?
- 8
Bien vu !
Bra upptäckt!
- 9
J'avais oublié ce cas-là.
Jag hade glömt det fallet.
- 10
T'as testé les cas limites ?
Har du testat kantfallen?
- 11
Les tests passent pas.
Testerna går inte igenom.
- 12
J'ai fait les modifs.
Jag har gjort ändringarna.
- 13
Tu peux y rejeter un œil ?
Kan du kika på det igen?
- 14
Pour moi c'est bon, j'approuve.
Det ser bra ut för mig, jag godkänner.
- 15
Tu peux merger.
Du kan merga.
- 16
Ça va casser l'API.
Det här kommer att förstöra API:et.
- 17
On traitera ça dans une autre PR.
Vi tar det i en annan PR.
- 18
C'est pas très lisible.
Det är inte särskilt lättläst.
- 19
C'est une question de goût.
Det är en smaksak.
- 20
Effectivement, t'as raison.
Det stämmer, du har rätt.