npm scripts and npm 3 support

Zenika-Training / zenika-formation-framework

Tools to build and present our training material

8 stars 25 forks source link

npm scripts and npm 3 support #64

Closed hgwood closed 9 years ago

hgwood commented 9 years ago

Deux éléments dans cette PR :

Pour que les formations puissent utiliser les npm scripts plutôt que d'avoir à installer Grunt, le framework ramènent maintenant grunt et grunt-cli comme dépendances.
- Les formations n'ont plus à dépendre de grunt et peuvent utiliser la commande grunt dans leurs npm scripts.
- Suite à cette PR, il faudrait tagguer une nouvelle version afin que les formations puissent avoir accès à cette feature. Je ferais un PR pour upgrader le modèle dès que ce sera fait.
Pour que le framework soit compatible avec npm 3, il ne suppose plus que les dépendances sont installées en arborescence (npm 3 installe tout à plat).
- J'en ai profité pour nettoyer un peu le code qui tourne autour des paths dans le Gruntfile
- J'en ai profité pour éliminer le mélange single/double quotes dans le Gruntfile

ldez commented 9 years ago

as tu testé cette branche avec le modèle ?

ldez commented 9 years ago

npm 3 est toujours en béta, je trouve risqué de migrer le framework pour son utilisation.

hgwood commented 9 years ago

@ldez

Oui j'ai testé avec le modèle.
La compatibilité avec npm 2 est maintenue. En fait j'ai juste retiré le fait que le fonctionnement du framework dépende de la façon dont npm arrange les modules dans node_modules. C'est une meilleure robustesse dans tous les cas.

ggerbaud commented 9 years ago

Arf... j'avais quand même fait beaucoup de truc cra-cra :mask:

Ok pour la compat 2-3, sinon par curiosité comment ça se passe pour quelqu'un qui mettrait a jour son npm. Npm3 remet automatiquement la hiérarchie à plat ?

hgwood commented 9 years ago

@ggerbaud T'inquiète, c'est pas possible de pas faire cra-cra du 1er coup !

Pour ta question, je viens de tester et la réponse est oui : si tu upgrades npm et que tu fais npm install alors que tu as déjà un node_modules installé par npm 2, tout est à plat à la fin.

ldez commented 9 years ago

@hgwood j'ai vu bien tes modifications, j'avais surtout peur sur le modèle avec NPM2 mais si tu as testé et que ça marche c'est top.

ggerbaud commented 9 years ago

@hgwood : ok. Est ce que tu fais la partie framework de Zenika/Formation--Modele#42 dans cette PR ou je peux merger et tu en prépares une autre ?

hgwood commented 9 years ago

Je ferais une autre PR pour Zenika/Formation--Modele#42

ggerbaud commented 9 years ago