Solify-IT / docs

0 stars 0 forks source link

Propuesta para crear un Pull Request #14

Closed Topi99 closed 3 years ago

Topi99 commented 3 years ago

P12: Proceso para crear un Pull Request

Objetivos

  1. Crear una solicitud de incorporación de código entendible por humanos y máquinas.

Involucrados

  1. Todo el equipo de desarrollo

Entradas

  1. Cambios en el código
  2. Necesidad de revisión de código.

Etapas y descripciones

Etapa Actividad Encargado Practica del CMMI
Creación del PR Asigna un nombre al PR, siguiendo el siguiente formato `CC>//`. Este formato permite tener un versionamiento semántico. Desarrollador asignado del cambio VER1.1, CM2.1
Llenar todos los campos (o los que apliquen) de la descripción del PR, para así dar más información a los reviewers del cambio. VER1.1, CM2.1
Asignar al encargado del PR (o a si mismo si es que aplica). VER2.1, CM2.1
Seleccionar encargados de la revisión (reviewers), o dejar que Github asigne automáticamente. VER2.1, CM2.1
Seleccionar el tab "state:ready-for-review". VER2.1, CM2.1
En la descripción del PR, añada preguntas para sus reviewers. También, si aplica, personalice los elementos del checklist al final de la descripción. VER1.3, CM2.1
Creación de entorno de prueba Decidir si se necesita un entorno de prueba en la nube o solamente descargar la rama con los cambios y probar localmente. Si se necesita un entorno en la nube, en el pipeline actual de Heroku crear dos nuevas aplicaciones: una para backend y otra para frontend. Apoyarse de su dueño de arquitectura para tener la configuración adecuada o siga el proceso para crear un entorno de verificación (pendiente). VER1.2, CM2.1
Notificar en el medio de comunicación adecuado que se ha creado un nuevo PR y que está listo para revisión. Así mismo, notificar si es necesario o no el entorno de pruebas en la nube. VER2.1, CM2.1

Salidas

  1. Pull Request completo
  2. Entorno de pruebas (ya sea en la nube o local).

Referencias

Conventional Commits V1.0.0 (CC)

Topi99 commented 3 years ago

@Solify-IT/wiki-owners

albertcastaned commented 3 years ago

Checklist

Notas

Lo veo muy bien. Solo me quedo duda con la etapa 1.1 el formato de los títulos de PR. ¿Se seguira la guia de commits o seria otro formato? Podría servir un ejemplo.

@Topi99

Topi99 commented 3 years ago

Lo veo muy bien. Solo me quedo duda con la etapa 1.1 el formato de los títulos de PR. ¿Se seguira la guia de commits o seria otro formato? Podría servir un ejemplo.

Sí, se sigue el mismo formato.

Topi99 commented 3 years ago

La sub practica SG2.1 "Seguir las peticiones de cambio" creo que aplica en todo el proceso.

¿Es VER2.1?

Topi99 commented 3 years ago

Ya está arreglado, solo me queda duda sobre si es VER2.1 o SG2.1.

albertcastaned commented 3 years ago

Perdon, me referia a la práctica SG2.1 de CM. Sería CM2.1 junto con VER2.1.

@Topi99

Topi99 commented 3 years ago

Ahh vale, vale, ahorita lo cambio.

albertcastaned commented 3 years ago

👍

AdrizPaola commented 3 years ago

Está súper bien!! :D