Skip to content

Conversation

@AlexisGauthierAtKronos
Copy link
Contributor

Proposition pour ajouter la possibilité d'utiliser le gestionnaire de dépendences Poetry en plus de Pipenv.

Le fichier python-pipenv.yml devient donc python-quality-checks.yml. C'est un breaking change, il faudra passer à la version v4.

On vient ici ajouter une petite recherche dans .tool-versions afin de voir si le gestionnaire 'poetry' est présent. Ensuite, selon le cas, on vient substituer les appels pour lancer les checks à l'aide de pipenv run ... ou bien de poetry run ...

J'ai fait l'essai en pointant sur la branche P22-802 de setup-python et ça fonctionne tant pour pipenv que poetry.

Copy link
Contributor

@kbeaulieu kbeaulieu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

equisoft-actions/setup-python pourrait output le dependency manager. On aurait pas besoin de la job dependencies-manager.

@AlexisGauthierAtKronos
Copy link
Contributor Author

equisoft-actions/setup-python pourrait output le dependency manager. On aurait pas besoin de la job dependencies-manager.

Ça me semble avoir bien du bon sens. Je vais essayer d'updater P22-802 pour obtenir l'output.

@AlexisGauthierAtKronos
Copy link
Contributor Author

equisoft-actions/setup-python pourrait output le dependency manager. On aurait pas besoin de la job dependencies-manager.

Ça me semble avoir bien du bon sens. Je vais essayer d'updater P22-802 pour obtenir l'output.

L'update a été effectué. python-workflows reçoit la chaîne pipenv ou poetry à partir de l'output de setup-python.

@AlexisGauthierAtKronos AlexisGauthierAtKronos merged commit 0a8c084 into main Jan 18, 2023
@AlexisGauthierAtKronos AlexisGauthierAtKronos deleted the P22-803 branch January 18, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants