← Back to issue list

build: use tox for tests

View original Github issue

Metadata

Project
charmcraft
Number
#1423
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged after dual approval. Replaced existing test configuration with tox, shifting Python version management to the tool while keeping the CI matrix OS-focused. Streamlined testing setup and reduced configuration lines.

Suggested action:

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after dual approval. Replaced existing test configuration with tox, shifting Python version management to the tool while keeping the CI matrix OS-focused. Streamlined testing setup and reduced configuration lines.
qwen3.6-35b-a3b-mtp-q6 Merged. Updated the CI test matrix to run per-OS, delegating Python version management to tox. Maintainer lengau submitted the change and requested the merge, which was approved.
qwen3.6-35b-a3b-mtp-q6 Merged after updating the test matrix to per-OS configurations and implementing tox for Python version management. The maintainer streamlined the CI workflow and requested the merge following these changes.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#616 build: use tox for unit tests in CI craft-parts merged Merged. CI configuration updated to use tox for unit tests. Approved by two reviewers with passing checks. Modified two files (+66/-10).
81%
#576 build: fix tox test environments craft-parts merged Merged a one-file change fixing tox test environments in the build configuration. Approved by two reviewers with zero unresolved comments. The update adjusted three lines to resolve environment setup issues.
76%
#4046 tools: move to the starbase tox configuration snapcraft merged Merged into main. Migrates tools to the starbase tox configuration. Approved by a reviewer, passes CI, and increases coverage by 2.88%. Updates the test matrix to include Python 3.8 and 3.10 on Ubuntu 20.04 and 22.04.
76%
#89 tox/ci: update Python versions tested starbase merged Merged after approval and passing CI checks. Updated tox and CI configuration to test newer Python versions. The change modified two files with a net reduction of one line. Code coverage remained at 100%.
76%
#32 tools: improve coverage support in the tox test environments starbase merged Merged improvements to coverage support in tox test environments. The change modifies two files and was approved by one reviewer prior to integration.
75%
#4036 tests: ensure coverage is set per tox env snapcraft merged Merged a single-file change (+3/-2 lines) to set test coverage per tox environment. Approved by a reviewer, passed CI, and integrated without changing coverage metrics.
74%
#4058 tox: fix linters and makefile snapcraft merged Merged into main after approval and passing CI. Fixes tox linter configurations and updates the Makefile, with no impact on test coverage.
74%
#4273 docs: fix tox doc builds snapcraft merged Merged to main after reviewer approval and passing CI. The change fixes tox documentation builds by modifying a single file (+2/-12 lines), with no impact on code coverage.
73%
#106 tox: improve UX starbase merged Merged into main following reviewer approval and passing CI. Resolves #105 by improving tox UX across three files. Changes maintain 100% test coverage with no impact on overall metrics.
73%
#4005 Testing: Lint and test using Tox snapcraft merged Merged after two approvals and passing CI. Configured Tox for linting and unit testing with a new GitHub Actions workflow. Legacy testing remains temporarily for comparison. Code coverage unchanged.
73%