search

LuisJal / ProyectoTienda

1 stars 1 forks source link

No es necesario node_modules #2

Open LuisPalacios opened 1 year ago

LuisPalacios commented 1 year ago

Creo que no es necesario node_modules ni package-lock.json

LuisJal commented 1 year ago

Claro que es necesario, sino no sabe que dependecias tengo que importar

LuisJal commented 1 year ago

Hace que sea mas comodo en vez de importarlo desde web, ya viene embebido en el codigo

LuisPalacios commented 1 year ago

de hecho es una discusión apasionada, mira

https://stackoverflow.com/questions/11459475/should-i-check-in-folder-node-modules-to-git-when-creating-a-node-js-app-on-he

Mi conclusión es que para Apps completas en producción basadas en NPM que "no" quieres que se te rompan y que subes a GitHub, lo lógico es que la acompañe un node_modules probado por el desarrollador, es decir incluirlo y NO bloquearlo en el .gitignore.

En este caso ignora el pull request pero sí que estaría bien que añadas este .gitignore para los que desarrollamos en Mac: 

# Generados por el S.O.
.DS_Store*
._.DS_Store*
ehthumbs.db
Icon?
Thumbs.db
*~
\#*\#
*.autosave

Siguiendo con el hilo, para Apps como esta de pruebas de concepto, laboratorio, aprendizaje creo que es mejor quitar rpm-modules, para hacerlo más ligero y que cada desarrollador se instale las librerías y dependencias.