search

Zenika-Training / zenika-formation-framework

Tools to build and present our training material
8 stars 25 forks source link

feat: gruntfile compatible avec npm 2 et 3 #115

Closed hgwood closed 8 years ago

hgwood commented 8 years ago

Cf #77. C'est pas particulièrement beau ou performant, mais je pense que c'est mieux que 2 builds différents.

J'ai aussi modifié les chemins qu'expose le serveur pour que index.html n'est pas à supporter des chemins différents pour npm 2 et 3.

jlandure commented 8 years ago

LGTM

ldez commented 8 years ago

du coup je pense que tu peux enlever ce que j'avais fait avec Gulp et mettre à jour le readme du modele.

ldez commented 8 years ago

Pour information, mon objectif était d'amorcer une migration vers Gulp.

hgwood commented 8 years ago

@ldez Mon but n'était pas d'évincer gulp :). En fait, j'ai voulu travailler sur #111 (migration vers Grunt 1) et je me suis rendu compte que travailler avec npm 2 n'allait pas être pratique pour moi donc j'ai regardé s'il n'y avait pas un moyen rapide de rendre le gruntfile compatible npm 3.

Il se trouve que par ailleurs, je suis maintenant sceptique sur une migration vers Gulp, vu la stagnation de son développement (la version 3 est obsolète mais l'équipe ne veut plus la maintenir, alors que la version 4 n'est pas terminée). Je serais plutôt en faveur de scripts npm. Cf #117.

hgwood commented 7 years ago

Cette modification a accidentellement cassé tous les builds car elle n'est pas compatible avec Node.js 0.10, utilisé sur Circle CI. Pourquoi elle n'est pas compatible ? Je ne sais pas. Mais on va dire que ce n'est pas grave car Node.js 0.10 est obsolète.