Enlève le chargement de nvm dans deploy.sh

[sandhose]

Q	R
Correction de bugs ?	Peut-être
Nouvelle Fonctionnalité ?	non
Tickets (issues) concernés

Etant donné que node à été réinstallé sur la préprod (via un dépot APT), et bientôt sur la prod, mais sans nvm, il ne faut plus le charger lors des déploiement. Ca devrait stabiliser un petit peu plus l'install 👍

[Situphen]

Euh en quoi ça va stabiliser l'installation ?

[sandhose]

Déjà, node est accessible sans activer nvm ; ensuite, y'avait des symlinks de node, npm, nvm, bower et gulp dans /usr/local/bin qui étaient cassés (en tout cas sur la preprod). Aussi, du fait que ça utilise le dépot de nodesources, on a toujours la dernière version stable de node via apt

[Situphen]

y'avait des symlinks de node, npm, nvm, bower et gulp dans /usr/local/bin qui étaient cassés (en tout cas sur la preprod).

C'est pas à cause de nvm (enfin, je pense pas...)

Aussi, du fait que ça utilise le dépot de nodesources, on a toujours la dernière version stable de node via apt

Est-ce mal ?

---

Après, si vous voulez enlever nvm, allez-y, peut-être que ça stabilisera l'installation. Même si je pense que mettre les dépendances en strict sera plus efficace...

[sandhose]

nvm est clairement pas fait pour la prod, car on a pas besoin de plusieurs
versions de node, et que de toutes façons, la où il était installé, ça
n'évitait pas de lancer certaines commandes en root... Donc, aucun intérêt,
d'autant plus que ça rend node inaccessible si on charge pas nvm

Le mar. 17 févr. 2015 00:44, Situphen notifications@github.com a écrit :

y'avait des symlinks de node, npm, nvm, bower et gulp dans /usr/local/bin
qui étaient cassés (en tout cas sur la preprod).

C'est pas à cause de nvm

Aussi, du fait que ça utilise le dépot de nodesources, on a toujours la
dernière version stable de node via apt

Est-ce mal ?

—
Reply to this email directly or view it on GitHub
#2351 (comment)
.

[SpaceFox]

Il manque surtout les instructions de migration (ce que tu as fait sur la préprod pour que je puisse le refaire sur la prod).

[SpaceFox]

BTW elle n'est pas à faire sur la branche de release cette PR ?

[landscape-bot]

Repository health decreased by 0.41% when pulling a7bbc79 on deploy-remove-nvm into 7ac5d90 on dev.

• 52 new problems were found (including 0 errors and 4 code smells).
• 4 problems were fixed (including 0 errors and 4 code smells).

[GerardPaligot]

Ca va faire presqu'un mois que cette PR est sans réponse et sans modification de @sandhose. Si je comprends bien ce qu'il reste à faire :

• @sandhose doit rédiger les instructions pour la migration
• @SpaceFox nous dire si c'est à faire sur la branche de prod ou de dev.

[SpaceFox]

@SpaceFox nous dire si c'est à faire sur la branche de prod ou de dev

Non, puisqu'on a MEP depuis...

[landscape-bot]

Repository health decreased by 0.55% when pulling a7bbc79 on deploy-remove-nvm into c827d7e on dev.

• 70 new problems were found (including 1 error and 14 code smells).
• 21 problems were fixed (including 0 errors and 12 code smells).

[pierre-24]

Vu #2455 ... Est ce que cette PR à encore du sens ?

[sandhose]

Non, elle peut être fermée

Le jeu. 2 avr. 2015 20:30, Pierre Beaujean notifications@github.com a
écrit :

Vu #2455 #2455 ... Est ce
que cette PR à encore du sens ?

—
Reply to this email directly or view it on GitHub
#2351 (comment)
.