Il y a une raison pour avoir mis la GPL2 spécifiquement ? Le client a demandé du dev libre ?
Ce n'est pas l'URI du plugin mais de Digitanie que tu as mis là. On n'a pas de site dédié au plugin, du coup pas vraiment d'intérêt de rajouter cette ligne.
idem commentaire précédent sur la licence
leaflet est déjà présent dans le projet ? il ne faut pas l'installer d'une manière ou d'une autre ?
C'est nécessaire ça du coup ? J'avais l'impression que ça ne changeait rien quand j'avais testé.
J'ai juste mis quelques questions, mais le code me convient!
Si je comprend bien, c'est plutôt dist/style.css pour le href non ?
Le fichier style.css est la version CSS du less de uikit? Si oui on ne veut pas forcément le versionner puisqu'on peut le récupérer en transpilant le less de la dépendance, avec la commande npm run build-css.
lui on veut bien le versionner pour pouvoir faire npm ci et non npm install justement. La différence est que ci va installer les versions fixes des dépendances en regardant le package-lock.json alors que le install va essayer de mettre à jour certaines dépendances s'il y a des MAJ et va mettre à jour le package-lock.
Pour moi c'est bon :) juste des petites remarques mais rien d'alarmant.
Ca c'est risqué, si tu ajoutes une entrée dans FOLDERNAMES ton indice ne sera plus bon
En utilisant npm et la configuration avec le package.json (https://docs.npmjs.com/cli/v10/configuring-npm/package-json?v=true) il ne sera plus nécessaire d'ignorer ces dossiers dans le versionneme…
UIKit est disponible sur npmjs (c.f. https://www.npmjs.com/package/uikit ) je pense que ça serait mieux de versionner le package.json et le package-lock.json avec la dépendance dedans plutôt que de le télécharger et de l'installer manuellement.