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] Rendre le script npm run clean vraiment utile en supprimant le répertoire dist et pas node_modules #460

Merged
merged 1 commit into from
Jan 6, 2023

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Jan 5, 2023

🎄 Problème

npm run clean ne supprime pas le répertoire dist alors que c'est un répertoire généré et que justement lorsqu'on teste on souhaite pouvoir supprimer tous les fichiers générés. Et par ailleurs le répertoire node_modules est, lui, supprimé alors que cela n'a non seulement aucune valeur ajoutée car on peut faire la même chose avec la commande npm ci mais qu'en plus c'est nuisible puisqu'on se retrouve avec un projet inutilisable alors qu'on voulait uniquement supprimer les fichiers générés.

🎁 Proposition

npm run clean supprime tous les fichiers générés et uniquement les fichiers générés : .nuxt, dist

🌟 Remarques

RAS

🎅 Pour tester

  1. Exécuter npm run build:site et vérifier que les répertoires .nuxt et dist sont présents
  2. Exécuter npm run clean et vérifier que les répertoires .nuxt et dist ont été supprimés

@lego-technix lego-technix self-assigned this Jan 5, 2023
@pix-service
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles ici.

@lego-technix lego-technix changed the title fix: make clean npm script useful in real workflow by cleaning built … [TECH] Rendre le script npm run clean vraiment utile en supprimant le répertoire dist et pas node_modules Jan 5, 2023
@lego-technix lego-technix force-pushed the fix-clean-dist-and-not-node_modules branch from 4254ee8 to 726ecb2 Compare January 5, 2023 17:55
@pix-service-auto-merge pix-service-auto-merge force-pushed the fix-clean-dist-and-not-node_modules branch from 726ecb2 to 97e9b23 Compare January 6, 2023 10:22
@pix-service-auto-merge pix-service-auto-merge merged commit babe8f9 into dev Jan 6, 2023
@pix-service-auto-merge pix-service-auto-merge deleted the fix-clean-dist-and-not-node_modules branch January 6, 2023 10:23
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.

6 participants