diff --git a/WORKFLOW.md b/WORKFLOW.md index 1a4142d..97478eb 100644 --- a/WORKFLOW.md +++ b/WORKFLOW.md @@ -11,6 +11,13 @@ La rama `main` está protegida con las siguientes reglas: ### ✅ Pull Request Reviews +- **Aprobaciones requeridas**: 1 (UNA aprobación necesaria) +- **Descartar aprobaciones obsoletas**: ✅ Habilitado (si hay nuevos commits, se descartan aprobaciones previas) +- **Requiere aprobación del último push**: ❌ Deshabilitado +- **No requiere code owners**: ❌ Deshabilitado (cualquier colaborador puede aprobar) + +### ✅ Protecciones Adicionales +- **Enforce admins**: ❌ Deshabilitado (el owner/administrador puede hacer bypass de las reglas) - **Aprobaciones requeridas**: 2 (DOS revisores deben aprobar) - **Descartar aprobaciones obsoletas**: ✅ Habilitado (si hay nuevos commits, se descartan aprobaciones previas) - **Requiere aprobación del último push**: ✅ Habilitado @@ -65,6 +72,13 @@ gh pr create --title "Título del PR" --body "Descripción detallada" ### 5. Proceso de Revisión +1. **Asignar revisores**: Mínimo 1 persona debe revisar +2. **Revisor aprueba**: El revisor debe dar "Approve" +3. **Si hay cambios solicitados**: Realizar commits adicionales + - ⚠️ Las aprobaciones previas se descartarán automáticamente + - Necesitarás una nueva aprobación +4. **Merge**: Una vez con 1 aprobación, se puede hacer merge +5. **Owner/Admin**: El propietario o administrador puede hacer merge sin aprobaciones si es necesario 1. **Asignar revisores**: Mínimo 2 personas deben revisar 2. **Revisores aprueban**: Ambos revisores deben dar "Approve" 3. **Si hay cambios solicitados**: Realizar commits adicionales @@ -118,6 +132,11 @@ gh pr close 123 ⚠️ **IMPORTANTE**: No se puede hacer push directo a `main`. Todos los cambios DEBEN pasar por Pull Request. +✅ **1 APROBACIÓN REQUERIDA**: El merge se habilita después de que 1 revisor apruebe el PR. + +👑 **OWNER/ADMIN BYPASS**: El propietario o administradores del repositorio pueden hacer merge sin aprobaciones si es necesario. + +⚠️ **Nuevos commits = Nueva aprobación**: Si haces push de nuevos commits después de recibir aprobación, la aprobación se descarta y necesitarás una nueva. ⚠️ **2 APROBACIONES REQUERIDAS**: El merge solo se habilita después de que 2 revisores diferentes aprueben el PR. ⚠️ **Nuevos commits = Nuevas aprobaciones**: Si haces push de nuevos commits después de recibir aprobaciones, las aprobaciones se descartan y necesitarás 2 nuevas.