Agrega pre-commit hook al front

caverav / auditforge

AuditForge is a pentest reporting application making it simple and easy to write your findings and generate a customizable report.

https://auditforge.feriadesoftware.cl

MIT License

20 stars 0 forks source link

Agrega pre-commit hook al front #77

Closed jllanosg closed 2 months ago

jllanosg commented 2 months ago

Descripción

Agrega el pre-commit hook al front, utilizando lint-staged.

Motivación y Contexto

Para asegurar una mínima calidad en los commits y evitar errores.

¿Cómo ha sido probado?

Intentando hacer commit de un console.log(""), el hook evita esto ya que no está permitido por ESLint.

Capturas de pantalla (si es apropiado):

Tipos de cambios

[ ] Bugfix (cambio que no interrumpe el funcionamiento y que soluciona un problema)
[x] New feature (cambio que no interrumpe el funcionamiento y que añade funcionalidad)
[ ] Breaking change (corrección o funcionalidad que podría causar que la funcionalidad existente cambie)

Lista de verificación:

[x] Mi código sigue el estilo de código de este proyecto.
[ ] Mi cambio requiere una modificación en la documentación.
[ ] He actualizado la documentación en consecuencia.

jllanosg commented 2 months ago

La verdad no veo funcionalidad en esto, tampoco vi ningún commit que incluya la carpeta _ (típica de husky), porfa revisa bien que no tengas archivos extras al momento de probar que funciona

Cuando instalé todo no hablaba nada de esto pero aparentemente viene con un .gitignore que hace que no se haya subido, debería quitarlo?

caverav commented 2 months ago

La verdad no veo funcionalidad en esto, tampoco vi ningún commit que incluya la carpeta _ (típica de husky), porfa revisa bien que no tengas archivos extras al momento de probar que funciona

Cuando instalé todo no hablaba nada de esto pero aparentemente viene con un .gitignore que hace que no se haya subido, debería quitarlo?

Creo que sería razonable subir el husky.sh, no? Podrías mandar tu git status? Para saber los archivos que no están trackeados?

jllanosg commented 2 months ago

La verdad no veo funcionalidad en esto, tampoco vi ningún commit que incluya la carpeta _ (típica de husky), porfa revisa bien que no tengas archivos extras al momento de probar que funciona

Cuando instalé todo no hablaba nada de esto pero aparentemente viene con un .gitignore que hace que no se haya subido, debería quitarlo?

Creo que sería razonable subir el husky.sh, no? Podrías mandar tu git status? Para saber los archivos que no están trackeados?

Si, la verdad no me fijé en eso, no hace sentido q no este incluido por defecto al instalarlo

^ ignorar el package-lock.json

Subo el _ entonces?

jllanosg commented 2 months ago

Voy a hacerlo otra vez