Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

feat: gruntfile compatible avec npm 2 et 3 #115

Merged
merged 1 commit into from
Jun 22, 2016

Conversation

hgwood
Copy link
Contributor

@hgwood hgwood commented Jun 18, 2016

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
Copy link
Contributor

LGTM

@ldez
Copy link
Contributor

ldez commented Jun 20, 2016

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

@ldez
Copy link
Contributor

ldez commented Jun 20, 2016

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

@hgwood
Copy link
Contributor Author

hgwood commented Jun 20, 2016

@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.

@ggerbaud ggerbaud merged commit c7c5953 into Zenika-Training:master Jun 22, 2016
This was referenced Jul 13, 2016
Closed
@hgwood hgwood deleted the grunt_upgrade branch July 14, 2016 20:02
@hgwood
Copy link
Contributor Author

hgwood commented Oct 13, 2016

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants