Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Faire évoluer le script de configuration #11475

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

theotime2005
Copy link
Contributor

@theotime2005 theotime2005 commented Feb 21, 2025

🥞 Problème

Dans le script de configuratoin, on fait plein de cd et de npm ci. De plus, on fait une première installation de certaines dépendances, puis on initialise la db, puis on installe les autres, bref c'est un peu désordonné.

🥓 Proposition

Dans l'ordre:

  • Installer toutes les dépendances avec npm run ci:all,
  • Initialiser la DB.
    On ajoute un argument qu'on peut passer à l'exécution du script "notvariables". Si cet argument est passé, alors le fichier .env ne sera pas écrasé.

🧃 Remarques

Il faudrait qu'au lieu d'être écrasé, le fichier .env soit complété avec les nouvelles variables d'environnement.

😋 Pour tester

  • Exécuter cette commande à la racine du mono-repo:
npm run configure -- notvariables
  • Constater que tout s'est bien passé,
  • Constater que le fichier .env dans l'api n'a pas été modifié.

@theotime2005 theotime2005 force-pushed the update-configure-script branch from 8f0e6c0 to c265702 Compare February 26, 2025 13:27
@theotime2005 theotime2005 force-pushed the update-configure-script branch from c265702 to 1e0284c Compare March 24, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants