Use Pipenv, and add requirements.txt for virtualenv usage #5

Eragon · 2019-04-25T15:04:31+02:00

Eragon commented

2019-04-25 15:04:31 +02:00

Pipenv est le remplaçant de virtualenv, normalement la configuration actuelle permet à un environnement de dev de fonctionner, via le Pipfile et le requirements.txt Pour l’environnement de prod il faut regarder les dépendance dans REQUIREMENTS.md

Résumé du GitLab

Lephé demandait à détailler. Ce que j'ai fait les, Python à moyen d'être géré dans des environnements virtuels et cette PR essaye d'ajouter le support correct des gestionnaires d'environnements virtuels. Il faut que je vire le fichier LICENSE(ou qu'on en choisisse une) idem pour les contributions Lephé avait aussi remarqué que j'avais des fichier de conf de la bdd, mais je ne sais pas ou ils sont, on peux me le dire ?

Si vous refusez l'ajout de ces fichiers je modifierais le gitignore pour qu'il ignore les fichiers des gestionnaires d'environnements virtuels

Pipenv est le remplaçant de virtualenv, normalement la configuration actuelle permet à un environnement de dev de fonctionner, via le Pipfile et le requirements.txt Pour l’environnement de prod il faut regarder les dépendance dans REQUIREMENTS.md ## Résumé du GitLab Lephé demandait à détailler. Ce que j'ai fait les, Python à moyen d'être géré dans des environnements virtuels et cette PR essaye d'ajouter le support correct des gestionnaires d'environnements virtuels. Il faut que je vire le fichier LICENSE(ou qu'on en choisisse une) idem pour les contributions Lephé avait aussi remarqué que j'avais des fichier de conf de la bdd, mais je ne sais pas ou ils sont, on peux me le dire ? Si vous refusez l'ajout de ces fichiers je modifierais le gitignore pour qu'il ignore les fichiers des gestionnaires d'environnements virtuels

Lephenixnoir commented

2019-05-03 10:50:12 +02:00

Alors, je suis chaud pour merger .env, .gitignore, Pipfile, REQUIREMENTS.md, requirements.txt tout de suite.

Je ne parlais pas de fichier de conf mais des scripts scripts/{init,migrate,run_dev}.sh que tu as peut-être poussés par erreur ?

Pour la license et le guide de contribution, j'aurais aimé qu'on en discute plus avant de le mettre sur le dépôt. 😃

Alors, je suis chaud pour merger `.env`, `.gitignore`, `Pipfile`, `REQUIREMENTS.md`, `requirements.txt` tout de suite. Je ne parlais pas de fichier de conf mais des scripts `scripts/{init,migrate,run_dev}.sh` que tu as peut-être poussés par erreur ? Pour la license et le guide de contribution, j'aurais aimé qu'on en discute plus avant de le mettre sur le dépôt. :smiley:

Eragon commented

2019-05-03 11:14:16 +02:00

Non non, ces scripts sont là exprès, ils sont spécifiques à pipenv, mais sont très utiles pour lancer plus facilement les commandes Je vais virer la license et le guide de contribution EDIT : C'est fait, j'ai viré la license et le guide de contributions

Lephenixnoir commented

2019-05-03 11:20:46 +02:00

Qu'entends-tu par nécessaire pour pipenv ? :o

Eragon commented

2019-05-03 11:35:15 +02:00

Ils sont pas nécessaires mais extrêmement utiles, c'est simplement des raccourcis de commandes.

Par exemple pipenv run {init,migrate,run_dev} exécutera le script spécifié au lieu de lancer une commande, c'est aussi un moyen d'ajouter quelques scripts d'automatisation, setup de la bdd, tests... etc

Ici init sert à initialiser la bdd(comme prévu selon le wiki), migrate quand à lui sert à effectuer les migrations de la bdd et run_dev lance le serveur flask avec les variables d'environnement pour avoir le débogueur. On pourra toujours en ajouter(par exemple un run_prod pour lancer le serveur flask sans les variables de débug)

Ils sont pas nécessaires mais extrêmement utiles, c'est simplement des raccourcis de commandes. Par exemple `pipenv run {init,migrate,run_dev}` exécutera le script spécifié au lieu de lancer une commande, c'est aussi un moyen d'ajouter quelques scripts d'automatisation, setup de la bdd, tests... etc Ici `init` sert à initialiser la bdd(comme prévu selon le wiki), `migrate` quand à lui sert à effectuer les migrations de la bdd et `run_dev` lance le serveur flask avec les variables d'environnement pour avoir le débogueur. On pourra toujours en ajouter(par exemple un `run_prod` pour lancer le serveur flask sans les variables de débug)

Lephenixnoir commented

2019-05-03 11:57:16 +02:00

Bon, j'achète. Je veux bien merge tout ça, puisque Darks n'a pas réagi je suppose que lui aussi. Comme les migrations échouent parfois à résoudre les diffs de schémas, on veut peut-être éviter les cas d'erreur :

Ajouter des && ou un mécanisme pour arrêter le script en cas d'erreur
Planter dans migrate.sh si aucun argument n'est fourni

La prochaine fois je serai moins chiant, c'est promis. xD

Bon, j'achète. Je veux bien merge tout ça, puisque Darks n'a pas réagi je suppose que lui aussi. Comme les migrations échouent parfois à résoudre les diffs de schémas, on veut peut-être éviter les cas d'erreur : * Ajouter des `&&` ou un mécanisme pour arrêter le script en cas d'erreur * Planter dans `migrate.sh` si aucun argument n'est fourni La prochaine fois je serai moins chiant, c'est promis. xD

Lephenixnoir approved these changes 2019-05-03 12:21:27 +02:00

Lephenixnoir closed this pull request

2019-05-03 12:21:52 +02:00

Sign in to join this conversation.

No reviewers