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

refactor: switch main package manager to Poetry. #48

Merged
merged 2 commits into from
Sep 10, 2024
Merged

Conversation

shahinism
Copy link
Collaborator

Pull Request Description

Summary:
This PR sets poetry as the default package manager for Python projects, providing seamless integration with devenv. pdm is not designed to run inside a virtual environment, which leads to various issues. To maintain backward compatibility, users can still opt to switch back to pdm if needed.

Details:

  • poetry is now the default package manager for Python due to its compatibility with devenv.
  • pdm is known to cause issues within virtual environments, so the initial approach to running it inside venv, is not optimal.
  • The option to use pdm remains available for those who prefer it.
  • Future support for pdm will be considered once devenv adds support (tracking issue: Support pdm cachix/devenv#703).

Why this change is necessary:

  • poetry works smoothly within devenv, whereas pdm introduces complications.
  • Ensuring a stable development environment while retaining flexibility for users who prefer pdm.
  • In the current version some functionalities of pdm don't work properly like pdm run ... or pdm remove ...

Next steps:

  • Once devenv adds support for pdm, we can re-evaluate its usage as the default package manager.

Tracking Issue:

@shahinism shahinism merged commit f921259 into main Sep 10, 2024
2 checks passed
@shahinism shahinism deleted the feat/python-poetry branch September 10, 2024 08:43
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.

1 participant