Revisando el repo me he dado cuenta de algunos aspectos a revisar:
Respecto a la creación de ramas; se debe contemplar una nueva rama con cada característica del código. Los test, al estar aplicando TDD forman parte del desarrollo de dicha característica y no tienen por qué estar en una rama a parte ya que una Pull Request debe ser auto-contenida y subir código que ya haya sido verificado, si se suben los test sin ningún código asociado no representa un hito para la rama principal. En resumen, los test se incluyen en cada característica y no en una rama a parte.
Los test deben tener la extensión .test.ts, por eso pedí que revisaran la configuración del repo antes de ponerse a programar.
Para ahorrar tiempo podemos aplicar una documentación de código más resumida y que no añade ruido al código programado que hasta ahora me han aceptado sin problema en la asignatura. Consta del siguiente formato:
Para las interfaces:
/**
* # Nombre De La Interfaz | Interface
* Brebe descripción del propósito
*
* ## Properties
*
* - lista
* - de
* - propiedades
*
* ## Methods
* - metodo() | `descripcion del metodo`
*
*/
Para las clases:
/**
* # Nombre De La clase | [Primary class / Secondary class] | extends ClasesQueExtiende | implements Interfaces, Que, Implementa
* Brebe descripción de la clase
*
* ## Properties
*
* - propiedad | descripción y tipo de dato
*
* ## Methods
*
* - metodo(parametros) | funcionamiento / valor de retorno
*/
Revisando el repo me he dado cuenta de algunos aspectos a revisar:
.test.ts
, por eso pedí que revisaran la configuración del repo antes de ponerse a programar.Para las interfaces:
Para las clases: