search

SocialGouv / reva

https://reva.beta.gouv.fr
Apache License 2.0
3 stars 3 forks source link

chore: root test script + nvmrc #584

Closed TitoVince35 closed 1 year ago

TitoVince35 commented 1 year ago

Deux ajouts mineurs pour plus de confort.

.nvmrc

Définit la version de node utilisée par le projet. Accessoirement ça permet de taper nvm use pour switcher directement dans la bonne version de node quand on utilise nvm ;)

Ajout du workspace reva-tests

Permet de définir un script npm run test à la racine du projet pour lancer les tests Cypress.

anthonny commented 1 year ago

Ok pour le nvmrc, bien que je sois surpris que nvm ne puisse pas s'appuyer sur "engines.node" du package.json Pour le workspace par contre, on ne va pas le faire pour le moment. Je ne sais pour quelles raisons obscures scalingo lors du npm ci installe quand meme les dev dependencies de reva-test et donc il installe cypress qui fait exploser la limite de taille de l'image docker autorisée par scalingo

C'est pour cette raison qu'il ne fait pas parti du workspace

cedricss commented 1 year ago

+1 pour utiliser engines.node, afin que ça fonctionne pour ceux qui utilisent d'autres outils (ex. n)

anthonny commented 1 year ago

J'ai trouvé cette issue https://github.com/nvm-sh/nvm/issues/651 On y parle de cette commande nvm use $(jq -r .engines.node package.json)

Du coup j'ai testé ca : nvm use $(cat package.json | npx json engines.node) qui fonctionne Ave n on a pas ce probleme car on a la commande n auto qui lit le engines.node

TitoVince35 commented 1 year ago

Lol!.. Bon ben j'oublie ma PR et je me mets à "n".