lgtm-com[bot]
commented
2 years ago This pull request introduces 1 alert when merging f99bc6723ece8af8b0af89b6b4152cfce58a4011 into fc3da5b1dc863e6cb07467b7916ab04534447b5c - view on LGTM.com
new alerts:
- 1 for Off-by-one comparison against length
Descripción
Algunas veces el script the feriados mostraba de forma incorrecta el siguiente feriado cuando el más cercano es un fin de semana. Aproveché para hacer un refactor para que sea más facil de entender ademas de un test básico (hacer tests con ava es horrible. -- para todos! 😡).
Por último, cuando el dia cuando se usa el script es un feriado, ahora va a mostrar el siguiente.
Ejemplo de comportamiento