ConceptoBloque 4 · Pull requests y forks
Revisar es corregir un examen
Explicación
Revisar un PR es como corregir un examen: no reescribes la hoja, marcas dónde hay que corregir y decides si pasa o no.
Todo se hace en Files changed, donde ves línea por línea los cambios:
- El + azul junto a una línea agrega un comentario en esa línea.
- Start a review agrupa tus comentarios y los publica juntos.
Tres formas de terminar una revisión
| Acción | Qué significa |
|---|---|
| Comment | Dejas comentarios sin tomar partido |
| Approve | Das el visto bueno. No hace el merge: es un paso aparte |
| Request changes | Pides cambios antes de integrar |
Tras un Request changes, el autor no abre otro PR: hace un commit nuevo en la misma rama, git push, y pide otra revisión. El autor no puede aprobar su propio PR.
Qué buscar al revisar
- Que lo que cambia coincida con la descripción del PR.
- Archivos o líneas que nadie mencionó.
- Contraseñas, llaves o llamadas raras.
- Marcadores de conflicto (
<<<<<<<) que se colaron.
Sugerir una línea
Con Add a suggestion propones cómo debe quedar una línea; el autor la acepta con Commit suggestion si está de acuerdo.