Skip to content
SP StackPractices
beginner

Plantilla de Pull Request

Plantilla de pull request completa para estandarizar code reviews y mejorar la calidad de merges.

Temas: devops

Resumen

Una plantilla de pull request estandariza la información provista al enviar cambios de código. Asegura que los revisores tengan contexto y que los autores verifiquen su trabajo antes de solicitar revisión.

Cuándo Usar

  • Tu equipo hace code reviews en cada cambio
  • Quieres reducir ida y vuelta en las reviews
  • Necesitas hacer cumplir estándares de testing o documentación
  • Manejas un proyecto open-source con colaboradores externos

Plantilla

## Descripción

[Descripción corta del cambio y su propósito]

Fixes # (issue)

## Tipo de Cambio

- [ ] Bug fix (cambio no breaking que corrige un issue)
- [ ] Nueva funcionalidad (cambio no breaking que agrega funcionalidad)
- [ ] Breaking change (fix o feature que rompe funcionalidad existente)
- [ ] Actualización de documentación
- [ ] Refactorización (sin cambios funcionales)
- [ ] Mejora de performance
- [ ] Actualización de dependencias

## Cambios Realizados

- [Cambio 1]
- [Cambio 2]
- [Cambio 3]

## Testing

- [ ] Tests unitarios agregados/actualizados
- [ ] Tests de integración pasan
- [ ] Testing manual realizado
- [ ] Casos edge cubiertos

### Evidencia de Testing

[Incluye screenshots, logs o comandos usados para testing]

## Checklist

- [ ] El código sigue las guías de estilo del proyecto
- [ ] Autoreview completado
- [ ] Comentarios agregados para lógica compleja
- [ ] Documentación actualizada (si aplica)
- [ ] Sin nuevas warnings o errores introducidos
- [ ] Pipeline CI/CD pasa

Secciones de la Plantilla

SecciónPropósito
DescripciónContexto para los revisores
Tipo de CambioCategoriza el PR
Cambios RealizadosLista de modificaciones
TestingEvidencia de que los cambios funcionan
ChecklistAutoverificación antes de review

Buenas Prácticas

  • Manténlo conciso: Plantillas largas desaniman completarlas
  • Usa checkboxes: Fáciles de escanear, difíciles de olvidar
  • Enlaza issues: Siempre referencia tickets relacionados
  • Incluye screenshots: Para cambios UI, la prueba visual es esencial
  • Automatiza donde sea posible: Deja que CI verifique lo que los bots pueden

Errores Comunes

  • Plantillas vacías: Enviar sin completar las secciones requeridas
  • Tests faltantes: Olvidar actualizar o agregar tests
  • Sin enlaces a issues: Hace más difícil rastrear el contexto

Preguntas Frecuentes

Cada pull request debería usar una plantilla?

Sí. Las plantillas aseguran que los reviewers obtengan contexto consistente y que los autores verifiquen su trabajo. Incluso fixes pequeños se benefician de una breve descripción y confirmación de testing.

Qué tan detallada debería ser la sección de testing?

Incluye suficiente detalle para que un reviewer pueda reproducir tus tests. Para cambios UI, adjunta screenshots o GIFs. Para cambios de API, incluye requests y responses de ejemplo.

Qué pasa si una plantilla de PR se siente muy pesada para mi equipo?

Empieza con una plantilla mínima: descripción, tipo de cambio y un checklist de 3 items. Expande secciones solo cuando notes brechas de información en las reviews.