search

DjLeChuck / recalbox-manager

A web interface to manage recalbox configuration
25 stars 10 forks source link

Update package.json #42

Closed substring closed 7 years ago

substring commented 7 years ago

Add jquery as an explicit dependency

fix #41 même si je n'ai pas pu tester le resultat. Mais au moins ca compilait

DjLeChuck commented 7 years ago

Ça compilerait oui.

Mon but est d'effectivement essayer d'utiliser la dépendance jQuery de NPM et ne plus utiliser une dépendance de fichier géré manuellement, mais je dois réglé des conflits avant rencontrés avec certains plugins.

MikaXII commented 7 years ago

Après si tu ne veux pas te faire trop chier aussi sur la gestion des dépendances enlève le '^' avant chaque numéro de version. Comme ça tu restes à des versions fixe et pas de mauvaises surprise.

DjLeChuck commented 7 years ago

Corrigé via b7c9560df223c019ebe2522ea0d4cf634103d224

J'ai confondu, c'est pour un projet du taf que j'ai des soucis, pas ici donc ta PR aurait suffit.

substring commented 7 years ago

Cela dit, le commentaire de @MikaXII reste tout à fait pertinent : on n'est pas à l'abri que ca repète un jour parce qu'une dépendance aura changé de version et pètera encore un truc

DjLeChuck commented 7 years ago

Si les paquets respectent le versionning semantic il n'y a pas de problème avec l'utilisation du caret plutôt que du tilde (ou de rien) donc je laisse comme c'est pour le moment.