Code Review with a French Colleague
Give and take code-review comments in French: soften a nitpick, ask why, push back politely and approve the merge.
French developers tutoient each other and do code reviews in a mix of French and English jargon: une PR, merger, un commit, les tests passent. Comments are often softened with the conditional (je renommerais) or with 'petit détail' for a nitpick, and 'c'est pas bloquant' signals you can merge anyway. Written comments stay a bit more careful than spoken ones, but the tone is direct and friendly.
- 1
Petit détail : je renommerais bien cette variable.
Small nitpick: I'd rename this variable.
- 2
C'est juste une suggestion, c'est pas bloquant.
It's just a suggestion, it's not blocking.
- 3
Je comprends pas trop ce que fait cette fonction.
I don't really get what this function does.
- 4
Tu pourrais ajouter un commentaire ici ?
Could you add a comment here?
- 5
Ça pourrait être simplifié.
This could be simplified.
- 6
Y a pas mal de code dupliqué.
There's quite a lot of duplicated code.
- 7
Pourquoi t'es parti sur cette approche ?
Why did you go with this approach?
- 8
Bien vu !
Good catch!
- 9
J'avais oublié ce cas-là.
I'd forgotten that case.
- 10
T'as testé les cas limites ?
Have you tested the edge cases?
- 11
Les tests passent pas.
The tests aren't passing.
- 12
J'ai fait les modifs.
I've made the changes.
- 13
Tu peux y rejeter un œil ?
Can you have another look?
- 14
Pour moi c'est bon, j'approuve.
Looks good to me, I'm approving.
- 15
Tu peux merger.
You can merge.
- 16
Ça va casser l'API.
This is going to break the API.
- 17
On traitera ça dans une autre PR.
We'll deal with that in another PR.
- 18
C'est pas très lisible.
It's not very readable.
- 19
C'est une question de goût.
That's a matter of taste.
- 20
Effectivement, t'as raison.
Fair point, you're right.