Correcciones y consejos

[Liluminalia]

Buenas Ivan! esta genial me encanta el readme, muy bien hecho, los test dan correcto y en general esta muy bien, lo único un par de cosas para mejorar:

• en los test, en el texto que pones en los describe, te recomendaría que pusieras el nombre de la función que estas testeando en vez de ponerlo en un comentario, te va a venir genial para detectar donde exactamente esta fallando el test cuando en un futuro próximo tengas que hacer muchos.
• hay que intentar no cargar los archivos con muchas funciones, hubiera estado mas perfecto aun si hubieras hecho un archivo por cada función, pero eso ya lo iras viendo también a lo largo del bootcamp, de momento tal y como esta, esta muy bien también.
• tengo una duda, no se muy bien porque, en algunas funciones, al declarar la variable como parámetro haces una copia, por ejemplo en arrayPush o en arrayUnshift, creo que en este tipo de funciones no haría falta, se hace mas para no mutar un estado o para desestructurar, intenta no complicar el código mas de lo que es necesario, ya daréis mas adelante los principios de diseño, como por ejemplo KISS, donde te explica que siempre que puedas mantengas un código simple, muchas veces nos enredamos y hacemos las cosas mas complicadas de lo que son, pasa mucho, intenta mantenerlo lo mas sencillo que te deje la función que estas realizando, lo que ayudara a la legibilidad de tu código, por ejemplo cuando trabajes en una empresa, trabajaras con un equipo, y todos deben poder entender tu código lo mas rápido posible, pero ya te digo todo esto son cosas que veréis mas adelante, esta muy bien en general.

sigue asi!! saludos!

[SemperFenix]

Buenas, Sara.

• De acuerdo, a partir de ahora pondré el nombre en Given the function arrayLength..., es verdad que es más descriptivo a la hora de leer el test, no lo pensé.

• De acuerdo, lo tendré en cuenta, me pareció más cómodo de esta manera en el ejercicio para poder llamar fácilmente a todas desde el test.

• En Push y en Unshift he pasado el parámetro desestructurado para que el método funcionen como los métodos originales , permitiendo pushear o unshiftear tantos parámetros como el usuario quiera; si no, sólo me permitía pushear o unshiftear un item.

Yo entendí en la documentación que con ...arg me convertía todos los valores que se pasen como argumento en un array, por eso lo pasé de esa manera.

¡Muchas gracias!

[Liluminalia]

en principio aun que estén cada una en un archivo, para eso se hace lo de module, para poder trabajar por módulos, asi que mientras estén exportadas deberías poder llamar a cada una igualmente, luego ya iras viendo que como buena practica lo suyo es que cada archivo tenga su test y tampoco todo en uno, pero ya te digo eso son detalles de momento, lo importante es que hacéis ya las ramas mejor y los commits etc, mas adelante ya iremos viendo.

animo!!

[SemperFenix]

¡Perfecto, así lo haré!

¡Muchas gracias!