P22-802 : Added option to setup dependencies with Poetry#10
Conversation
kbeaulieu
left a comment
There was a problem hiding this comment.
Et éventuellement on souhaite remplacer complètement pipenv par poetry?
J'aurais tendance à dire que oui. Mon impression est que Poetry est supérieur à Pipenv. Déjà, ça fait deux ou trois fois que j'ai du mal à résoudre l'arbre des dépendances avec Pipenv et qu'avec Poetry je réussi à peu près immédiatement. Les messages lors d'incompatibilités sont beaucoup plus clairs. Cela dit, les workflows pourront gérer les deux, je ne vois pas d'obligation pour tout changer vers Poetry. |
613aa6d to
acca3ee
Compare
|
J'ai updaté la PR pour offrir en output la chaîne 'pipenv' ou 'poetry'. |
Je propose d'ajouter l'option d'utiliser le gestionnaire de dépendances Poetry.
Le setup avec Pipenv reste disponible pour les projets qui l'utilise toujours.
Avant d'installer les dépendances, on vient détecter si Poetry est utilisé dans
.tool-versions. S'il est présent, on fait le setup pour Poetry, sinon on fait le setup pour Pipenv.J'ai fait des essais avec des branches utilisant Poetry et des branches utilisant Pipenv, tout semble fonctionnel.
Il ne resterait qu'à mettre à jour aussi le projet python-workflows pour utiliser Poetry. En attendant cette mise à jour Pipenv reste utilisable sans modifications aux autres projets, ce n'est pas un breaking change.