Switch from Nox to Tox for multi-version Python testing#550
Open
tkoyama010 wants to merge 4 commits intomainfrom
Open
Switch from Nox to Tox for multi-version Python testing#550tkoyama010 wants to merge 4 commits intomainfrom
tkoyama010 wants to merge 4 commits intomainfrom
Conversation
- Add tox.ini with support for Python 3.10, 3.11, and 3.12 - Define test environments: - py310, py311, py312 - Run tests with pytest - docs - Build documentation with Sphinx - lint - Run pre-commit hooks - Remove noxfile.py - Update GitHub Actions workflow to use tox
Contributor
There was a problem hiding this comment.
Pull request overview
This PR migrates the project's test automation from Nox to Tox, maintaining support for Python 3.10, 3.11, and 3.12 (dropping 3.9 support, which is consistent with the project's existing requires-python = '>=3.10' requirement in pyproject.toml).
Changes:
- Adds tox.ini configuration with test, docs, and lint environments
- Removes noxfile.py
- Updates GitHub Actions workflow to use tox with tox-gh-actions plugin
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| tox.ini | New tox configuration file defining test environments for Python 3.10-3.12, plus docs and lint environments |
| noxfile.py | Removed nox configuration that previously supported Python 3.9-3.12 |
| .github/workflows/testing-and-deployment.yml | Updated to install and run tox instead of directly installing test dependencies and running pytest |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Remove noxfile.py (switching to tox) - Update tox.ini to support Python 3.11, 3.12, and 3.13 (matching updated noxfile)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR replaces Nox with Tox for test automation, providing support for multiple Python versions.
Changes
Usage
Run all tests:
Run specific Python version:
Build docs:
Run linting:
Benefits