PULL REQUESTS Y CODE REVIEW
El corazón del trabajo en equipo: proponer cambios y revisarlos antes de mergear.
¿QUÉ ES UN PULL REQUEST?
Un Pull Request (PR) es una propuesta para integrar tu rama a main. Permite que tus compañeros REVISEN tu código antes de mergearlo.
Flujo típico: 1. Creas una rama y haces commits. 2. Pusheas la rama. 3. Abres un PR en GitHub. 4. Tus compañeros revisan, comentan, sugieren cambios. 5. Cuando aprueban, se mergea a main.
CÓMO CREAR UN BUEN PR
✅ Título claro: '[feature] Login con Google' ✅ Descripción: qué hace, por qué, cómo probarlo ✅ Screenshots si es UI ✅ Mantén el PR PEQUEÑO (menos de 400 líneas) ✅ Referencia el issue: 'Closes #42'
❌ NO mezcles features distintos en un PR ❌ NO subas archivos generados (build, node_modules)
Un PR pequeño se revisa rápido. Un PR enorme se revisa mal o nunca.
CODE REVIEW: REVISAR PRS DE OTROS
Cuando revises código de un compañero:
• Sé AMABLE pero HONESTO. • Pregunta antes de criticar: '¿por qué elegiste X?' • Distingue entre 'esto es un bug' y 'preferencia personal'. • Aprueba con 👍 o pide cambios con sugerencias concretas. • Si todo está OK, mergea o deja que el autor lo haga.