[TECH] Fixer la version NodeJS à 16.14.0

[nlepage]

🎄 Problème

La version de NodeJS est à 12.14.0 dans CircleCI et à 16.13.0 dans le package.json...

🎁 Solution

Fixer la version de NodeJS à 16.14.0, dans CircleCI et dans le package.json (même version que les apps).

🌟 Remarques

npm < 8.13.2 rencontre une erreur lors de l'installation des dépendances :

npm ERR! Invalid Version: 3.5.0pr2

Cette erreur se produit pendant la phase d'audit et vient de la dépendance yui.

• See [Bug] npm install breaks with npm version 8 storybookjs/ember-cli-storybook#124
• See [BUG] Invalid semver in package history causes crash when installing a package npm/cli#5017

On met donc npm à jour en npm v8.13.2.

🎅 Pour tester

Si la CI est au vert, c'est bon.

[pix-service]

I'm deploying this PR to:

• Storybook: https://ui-pr224.review.pix.fr

Please check it out!

[yannbertrand]

P'tite mise à jour du .nvmrc aussi pour passer en v16.14.0 ?

[yannbertrand]

D'ailleurs je me demande si .node-version est pas mieux que .nvmrc ? Mais je ne sais pas trop la différence

[nlepage]

D'ailleurs je me demande si .node-version est pas mieux que .nvmrc ? Mais je ne sais pas trop la différence

Il semble que .node-version ne soit pas reconnu par nvm, en revanche il est reconnu par asdf mais je ne suis pas sur qu'on soit très nombreux à l'utiliser...

[nlepage]

P'tite mise à jour du .nvmrc aussi pour passer en v16.14.0 ?

Done.

[yannbertrand]

npm 8.3.1 rencontre une erreur lors de l'installation des dépendances :

npm ERR! Invalid Version: 3.5.0pr2

Le problème vient de la dépendance [email protected], voire npm/cli#5017

Contournement mis en place avec l'option --no-audit, mais l'ennui c'est que ça va aussi impacter nos environnements de dev...

On a également testé avec une version plus récente de npm, mais le bug est toujours présent.

Est-ce qu'on a pas moyen de fixer le problème avec la propriété overides des package.json ? (dispo depuis [email protected])

[nlepage]

PR ouverte chez ember-cli-storybook : storybookjs/ember-cli-storybook#125

[nlepage]

Le problème a été corrigé en npm v8.13.2, il faut voir si on peut upgrade sur cette version.

[nlepage]

Ça fonctionne en faisant un upgrade de npm en v8.13.2, donc plus besoin de contournements !

Cependant sur les apps c'est npm v8.3.1 qui est attendu, donc les personnes qui travaillent sur pix-ui et sur les apps vont être gênées...

Je vais voir pour ouvrir une PR qui autorise npm ^v8.3.1 sur les apps...