Skip to content

Update workflow documentation - 1 approval required #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ajeelias wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update workflow documentation - 1 approval required #3

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c06e64a
Add Git workflow documentation for Pull Request process
ajeelias Nov 1, 2025
4668dfc
Update branch protection rules documentation
ajeelias Nov 1, 2025
fff1069
Merge branch 'main' into docs/update-workflow-rules
ajeelias Nov 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions WORKFLOW.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down Expand Up @@ -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.
Expand Down