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

Remove tox and use poe for task execution instead #867

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kevinkjt2000
Copy link
Contributor

Tox served us well, but it is something we have discussed replacing. The
ecosystem for version selection is better than it was; poetry now has
the ability to easily switch versions with poetry env. The format of
tox.ini is not as easy as poe's configuration in pyproject.toml;
this should simplify adding and maintaining tasks. Additionally, tox
would occasionally release breaking changes such as hiding environment
variables; switching to a simpler task execution command should help
minimize the amount of breaking changes because it is a smaller
codebase.

Tox served us well, but it is something we have discussed replacing. The
ecosystem for version selection is better than it was; poetry now has
the ability to easily switch versions with `poetry env`. The format of
`tox.ini` is not as easy as poe's configuration in `pyproject.toml`;
this should simplify adding and maintaining tasks. Additionally, tox
would occasionally release breaking changes such as hiding environment
variables; switching to a simpler task execution command should help
minimize the amount of breaking changes because it is a smaller
codebase.
@kevinkjt2000 kevinkjt2000 linked an issue Jul 28, 2024 that may be closed by this pull request
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.

Remove tox
3 participants