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

Use tox #1361

Closed
micahellison opened this issue Oct 23, 2021 · 0 comments · Fixed by #1504
Closed

Use tox #1361

micahellison opened this issue Oct 23, 2021 · 0 comments · Fixed by #1504
Labels
build Issues related to the build pipeline 📌 This can't go stale
Milestone

Comments

@micahellison
Copy link
Member

We're thinking about using tox for configuration and possibly simplifying our GitHub Actions pipelines. Switching to it could fix a few pain points:

  • quickly testing locally on a variety of Python versions
  • being able to test both locally and on GitHub Actions with mostly the same code
  • possibly being able to sidestep some environment/syntax issues we've had between *nix/bash and Windows/PowerShell on GH Actions
  • replacing the makefile, allowing Windows users to quickly use build automation without having to deal with Make for Windows (which hasn't been updated since 2006)

Poetry supports tox and has some notes on using it in its docs. We just merged in #1359 which resolves the pre-req it mentions by using poetry-core instead of poetry.

Feel free to chime in with pros/cons or any complications that might arise.

@micahellison micahellison added 🆕 New! build Issues related to the build pipeline labels Oct 23, 2021
@micahellison micahellison removed the 🆕 New! label Nov 6, 2021
@micahellison micahellison added this to the Backlog milestone Nov 6, 2021
@micahellison micahellison added the 📌 This can't go stale label Nov 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues related to the build pipeline 📌 This can't go stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant