Code-Review mit einem spanischen Kollegen
Mit einem spanischsprachigen Entwickler höflich Code-Review-Feedback geben und annehmen.
In spanischen Tech-Teams läuft das Code-Review über PR-Kommentare und kurze Calls, mit jeder Menge Englisch dazwischen: la PR, los tests, mergear. Kritik wird mit dem Konditional und kleinen Abschwächungen wie una tontería (eine Kleinigkeit) oder es solo una sugerencia (nur ein Vorschlag) abgefedert. Die Karten begleiten zwei Entwickler, die gegenseitig ihre Pull-Requests reviewen.

- 1
Una tontería: quizás cambiaría el nombre de esta variable.
Nur eine Kleinigkeit: Ich würde diese Variable vielleicht umbenennen.
- 2
Es solo una sugerencia, no es bloqueante.
Ist nur ein Vorschlag, nichts Blockierendes.
- 3
No acabo de entender qué hace esta función.
Ich verstehe nicht ganz, was diese Funktion macht.
- 4
¿Podrías añadir un comentario aquí?
Könntest du hier einen Kommentar ergänzen?
- 5
Esto se podría simplificar.
Das könnte man vereinfachen.
- 6
Hay mucho código duplicado.
Es gibt viel duplizierten Code.
- 7
¿Has tenido en cuenta el caso de que la lista esté vacía?
Hast du den Fall berücksichtigt, dass die Liste leer ist?
- 8
Bien visto, no lo había pensado.
Gut gesehen, daran hatte ich nicht gedacht.
- 9
Ahora lo arreglo.
Ich behebe es gleich.
- 10
Lo he hecho así a propósito.
Ich habe es absichtlich so gemacht.
- 11
Te explico por qué.
Ich erkläre dir, warum.
- 12
Vale, ahora tiene más sentido.
Okay, jetzt ergibt es mehr Sinn.
- 13
Están fallando los tests en el CI.
Die Tests schlagen im CI fehl.
- 14
En mi máquina funciona.
Auf meinem Rechner läuft es.
- 15
Preferiría dividir esto en dos PR.
Ich würde das lieber in zwei PRs aufteilen.
- 16
¿Le puedes echar otro vistazo cuando puedas?
Kannst du noch mal drüberschauen, wenn du Zeit hast?
- 17
Te he dejado unos cuantos comentarios.
Ich habe dir ein paar Kommentare hinterlassen.
- 18
He aplicado los cambios que me sugeriste.
Ich habe die Änderungen eingebaut, die du vorgeschlagen hast.
- 19
Por mí está bien, te la apruebo.
Von mir aus passt es, ich gebe es dir frei.
- 20
Ya la puedes mergear.
Du kannst ihn jetzt mergen. (den PR)