B2 · Intermedio alto · 6 lecciones
Code review en inglés
Pedir cambios, responder comentarios y defender tus decisiones. Estas son las frases, la gramática y los ejemplos que usarás de verdad, explicados en español.
Pedir cambios con tacto
La misma petición puede sonar a ayuda o a ataque. Depende de cómo la formules.
Frases clave
| Inglés | Español |
|---|---|
| Could we rename this variable to make it clearer? | ¿Podríamos renombrar esta variable para que quede más clara? |
| I'd suggest extracting this into a helper. | Sugeriría extraer esto a un helper. |
| What do you think about adding a test here? | ¿Qué te parece añadir un test aquí? |
Gramática · Peticiones suaves
Could we… · I'd suggest + -ing · What do you think about + -ing
En una review no das órdenes: propones. Estas fórmulas convierten “cambia esto” en una sugerencia.
I'd suggest using a map here.
Sugeriría usar un map aquí.
💡 Después de suggest y de about, el verbo va en -ing: I'd suggest using…
Así lo dirías
Sarah: You're reviewing Leo's PR. What would you say about his very long function?
Tú: This function is getting quite long. Could we split it into smaller ones?
“You must” suena a orden y además va sin “to”. Mejor describe el problema y propón con “Could we…?”.
Responder a comentarios
Cuenta qué has cambiado tras la review: el resultado importa más que el momento.
Frases clave
| Inglés | Español |
|---|---|
| Good catch, I've fixed it. | Bien visto, ya lo he arreglado. |
| I've pushed a new commit with the changes. | He subido un commit nuevo con los cambios. |
| I've left a comment explaining why. | He dejado un comentario explicando el porqué. |
Gramática · Present perfect
have / has + participio
Para contar algo que ya has hecho y cuyo resultado importa ahora, sin decir cuándo.
I've addressed all your comments.
He resuelto todos tus comentarios.
💡 Si dices cuándo (yesterday, this morning at 9), usa el pasado simple.
Así lo dirías
Sarah: I see you changed the cache logic. What did you change?
Tú: I've switched the cache to Redis and added a test for it.
Después de have va el participio: switched o changed, added. Y no hace falta repetir “I have”.
Defender tu decisión
Defiende tu código con argumentos, no a la defensiva.
Frases clave
| Inglés | Español |
|---|---|
| I did it this way because it's easier to test. | Lo hice así porque es más fácil de testear. |
| It's a bit slower, but it's much simpler. | Es un poco más lento, pero mucho más sencillo. |
| Although it adds a dependency, it saves us a lot of code. | Aunque añade una dependencia, nos ahorra mucho código. |
Gramática · Conectores para argumentar
because · so · but · although
Because da la razón, so la consecuencia, but y although el contraste.
I used a queue because the API has rate limits.
Usé una cola porque la API tiene límites de peticiones.
💡 Although va delante de una idea completa: Although it's slower, it's simpler.
Así lo dirías
Leo: Why didn't you use the existing helper?
Tú: Because the helper does too many things and it's hard to test.
Para algo que siempre es así, presente simple: does. Y “hard to test”, no “difficult for test”.
Proponer alternativas
Plantea escenarios y alternativas con condicionales.
Frases clave
| Inglés | Español |
|---|---|
| If we cache the response, the page will load faster. | Si cacheamos la respuesta, la página cargará más rápido. |
| If we had more time, I would rewrite this module. | Si tuviéramos más tiempo, reescribiría este módulo. |
| What would happen if the token expired? | ¿Qué pasaría si caducara el token? |
Gramática · Condicionales
if + presente → will · if + pasado → would
El primero para lo que es probable; el segundo para hipótesis o cosas poco probables.
If we had more time, we would add more tests.
Si tuviéramos más tiempo, añadiríamos más tests.
💡 Nunca will detrás de if: If it fails, we'll roll back.
Así lo dirías
Sarah: What would you do differently if you had another week?
Tú: If I had another week, I'd write more tests and improve the logging.
Es una hipótesis: if + pasado (had) → would (I'd).
Aprobar y cerrar la review
Los phrasal verbs que salen en todas las reviews.
Frases clave
| Inglés | Español |
|---|---|
| Feel free to go ahead and merge it. | Adelante, puedes mergearlo. |
| Let's follow up on this in a separate ticket. | Retomemos esto en un ticket aparte. |
| If something breaks, we can roll it back. | Si algo se rompe, podemos revertirlo. |
Gramática · Phrasal verbs del día a día
verbo + partícula (go ahead, follow up, roll back)
Muchos verbos cambian de significado con una partícula. En las reviews salen constantemente.
We can roll it back in two minutes.
Podemos revertirlo en dos minutos.
💡 Con un pronombre como objeto, va en medio: roll it back, no “roll back it”.
Así lo dirías
Leo: The PR is approved. Anything else before you merge?
Tú: No, I'll merge it now and follow up on the docs tomorrow.
“Follow up on” lleva on. Y con will no hace falta repetir el sujeto: I'll merge it and follow up…
Una review completa
La estructura de un comentario que se agradece.
Frases clave
| Inglés | Español |
|---|---|
| Nice approach, I like how clean this is. | Buen enfoque, me gusta lo limpio que queda. |
| nit: there's a typo in this comment. | Detalle menor: hay una errata en este comentario. |
| This could break if the list is empty. | Esto podría romperse si la lista está vacía. |
Gramática · Estructura de un buen comentario
lo positivo → el problema → la propuesta
Empieza por lo que funciona, explica el problema concreto y termina con una propuesta abierta.
Nice approach! This could fail on empty lists, could we add a guard?
¡Buen enfoque! Esto podría fallar con listas vacías, ¿podríamos añadir una comprobación?
💡 Prefijos útiles: “nit:” para detalles menores, “question:” si solo preguntas.
Así lo dirías
Sarah: Give me your review of this PR in one or two sentences.
Tú: The code looks good, but there aren't any tests yet. Could we add a few?
“There aren't any tests” (plural). Y en vez de “you need”, propón: Could we add…?