ES03-Estándar de Pull Requests
Los Pull Requests deben tener la menor cantidad de cambios como sea posible. Cambios con archivos autogenerados deben tener su propio Pull Request.
Los Pull Requests se deben abrir como borradores, con el prefijo
[WIP](work in progress), en cuanto se haga un primer commit.Los Pull Requests deben ser aprobados por al menos una persona externa a la funcionalidad siendo implementada. (Si dos personas están trabajando juntas en una misma funcionalidad, el Pull Request NO PUEDE ser aprobado por ninguna de ellas)
Para un Pull Request se debe usar el mismo nombramiento que en los Commits (ES02), añadiendo
[WIP]al principio para Pull Requests en borrador.La descripción de los Pull Requests debe incluir un mensaje con cualquier bloqueo (otros Pull Requests con cambios que sean necesarios para integrar el actual) y una lista con cambios por hacer en el siguiente formato:
[## ⛔ Waiting on PR #<PrNo>]
## Changes:
- [ ] Cambios en lista
[## Notes:]
- [ ] Notas Adicionales
Ejemplo
Título: [WIP] feat(auth): ✨ implement admin login logic
## ⛔️ Waiting on PR #420
## Changes
- [x] Change 1
- [ ] Change 2
- [ ] Change 3
Indica que se debe intergar el Pull Request #420, y que se debe esperar a que se hagan los cambios indicados en la lista. Este Pull Request deberá ser un borrador y estar marcado como [WIP] hasta que todos los cambios indicados hayan sido marcados.
Referencias
Autores
Versiones
| Versión | Descripción | Responsables | Fecha |
|---|---|---|---|
| 1.0 | Creación de la Política | Bernardo Estrada | 18/03/2022 |
| 1.1 | Corrección ortográfica | Bernardo Estrada | 23/03/2022 |
| 1.2 | Aumentar nivel de detalle | BE & MN | 04/04/2022 |
| 1.3 | Corrección ortográfica | Guillermo C. Espino | 06/04/2022 |
| 1.4 | Cambio a estándar | Alberto Matute | 03/05/2022 |
| 1.5 | Cambio de estilo de links | Alberto Matute | 07/06/2022 |