Done is better than perfect.
Esta lista puede ir cambiando con el tiempo y conforme al equipo
- La funcionalidad cumple los criterios de aceptación definidos en la historia de usuario (Pivotal Tracker || Waffle.io || GitHub Issue).
- Cuenta con pruebas de integración pasando en verde, y no rompe con otras pruebas ya existentes previamente.
- Los mensajes de los commits son descriptivos y en inglés. En caso de que sean demasiados
commits que no ayudan a mantener un historial descriptivo, usa
git squash
. - El feature se encuentra en un branch
feature/name-of-feature
y los cambios con commits. Este branch se encuentra por encima demaster
; en caso de que no, se puede hacer ungit rebase
. - El branch tiene un Pull-Request (PR) abierto a
master
y el botón de Merge está en verde. - El código del PR ha sido revisado por un colega del equipo, en caso de que no fue desarrollado en pair programming.
- El PR se subió al ambiente de pruebas (staging server). La historia de usuario cambia a "Needs revision" (en GitHub issues) o "Deliver" (en Pivotal Tracker).
- La funcionalidad fue aceptada por el Product Owner.
- El PR es aceptado por el Delivery Owner o Dev Lead del equipo.
- Realizar un deployment al ambiente de producción (production server).