-
Notifications
You must be signed in to change notification settings - Fork 0
Tox ini cleanup obsolete py27 test environments #18
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
Tox ini cleanup obsolete py27 test environments #18
Conversation
Signed-off-by: Bernhard Kaindl <[email protected]>
Signed-off-by: Bernhard Kaindl <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR removes obsolete Python 2.7 test environments and configuration from the tox setup. The changes update the minimum tox version requirement and clean up all Python 2.7 related dependencies, test environments, and documentation references.
- Removes all Python 2.7 specific test environments and dependencies
- Updates minimum tox version to 4.6 and simplifies development setup
- Cleans up obsolete coverage combination functionality that was specific to Python 2.7
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| tox.ini | Removes py27 environments, covcombine section, and updates configuration for modern tox |
| pyproject.toml | Adds test dependencies to coverage, mypy, and pytype extras |
| README.md | Removes Python 2.7 support references and simplifies CI description |
| CONTRIBUTING.md | Removes Python 2.7 development setup instructions and simplifies tox installation |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| "python-libs[test]", | ||
| "coverage[toml]", | ||
| "diff_cover" | ||
| ] | ||
| mypy = [ | ||
| "python-libs[test]", |
Copilot
AI
Aug 27, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding 'python-libs[test]' as a dependency to the coverage extra creates a circular dependency since this is the same package being defined in this pyproject.toml file. This should be removed or the test dependencies should be listed explicitly.
| "python-libs[test]", | |
| "coverage[toml]", | |
| "diff_cover" | |
| ] | |
| mypy = [ | |
| "python-libs[test]", | |
| "mock", | |
| "pyfakefs", | |
| "pytest>=7", | |
| "pytest-cov", | |
| "pytest-forked", | |
| "pytest_httpserver", | |
| "pytest-localftpserver", | |
| "pytest-subprocess", | |
| "pytest-timeout", | |
| "typing_extensions", | |
| "coverage[toml]", | |
| "diff_cover" | |
| ] | |
| mypy = [ | |
| "mock", | |
| "pyfakefs", | |
| "pytest>=7", | |
| "pytest-cov", | |
| "pytest-forked", | |
| "pytest_httpserver", | |
| "pytest-localftpserver", | |
| "pytest-subprocess", | |
| "pytest-timeout", | |
| "typing_extensions", |
| "diff_cover" | ||
| ] | ||
| mypy = [ | ||
| "python-libs[test]", |
Copilot
AI
Aug 27, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding 'python-libs[test]' as a dependency to the mypy extra creates a circular dependency since this is the same package being defined in this pyproject.toml file. This should be removed or the test dependencies should be listed explicitly.
| "types-toml", | ||
| ] | ||
| pytype = [ | ||
| "python-libs[test]", |
Copilot
AI
Aug 27, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding 'python-libs[test]' as a dependency to the pytype extra creates a circular dependency since this is the same package being defined in this pyproject.toml file. This should be removed or the test dependencies should be listed explicitly.
| "python-libs[test]", | |
| "mock", | |
| "pyfakefs", | |
| "pytest>=7", | |
| "pytest-cov", | |
| "pytest-forked", | |
| "pytest_httpserver", | |
| "pytest-localftpserver", | |
| "pytest-subprocess", | |
| "pytest-timeout", | |
| "typing_extensions", |
CONTRIBUTING.md
Outdated
| systemctl enable --now --user podman.socket # Only configures the podman socket | ||
| systemctl start --user podman.socket # Start the docker-compatible unix socket | ||
| # Ubuntu only, Fedora 37 configures it already with more unqualifies search registries: | ||
| # Ubuntu only, Fedora configures unqualified search registries out-of-the box: |
Copilot
AI
Aug 27, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a spelling error in 'out-of-the box' - it should be 'out-of-the-box' with hyphens.
| # Ubuntu only, Fedora configures unqualified search registries out-of-the box: | |
| # Ubuntu only, Fedora configures unqualified search registries out-of-the-box: |
…7 tests Signed-off-by: Bernhard Kaindl <[email protected]>
0400cc5 to
3590b8a
Compare
No description provided.