build: use tox for tests
Metadata
Current evaluation
Merged #1423 in commit 8b291de3, moving test execution to tox so tox handles Python versions while CI matrices cover operating systems.
Resolution: closed resolved
Reason: PR charmcraft#1423 was approved and merged as commit 8b291de3, completing the tox-based test workflow change.
Evaluated by: mr-cal-bot
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #1423 in commit 8b291de3, moving test execution to tox so tox handles Python versions while CI matrices cover operating systems. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit 8b291de3 (Dec 2023): charmcraft's own unit/integration tests now run via tox; the GitHub Actions matrix was reduced to per-OS, with tox handling Python versions instead. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1423 in commit 8b291de3, switching tests to tox. Tox now handles Python versions while the CI matrix is limited to operating systems. | |
| 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. |
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 PR #616 switched unit-test CI to tox; it was approved by two reviewers and landed as commit bd6c687f. | |
| #1674 fix: make tox tests succeed | charmcraft | merged | Merged in commit b24ccd5b: fixed the tox environment variable and added a workaround for canonical/operator#1212, allowing tox tests to succeed. | |
| #89 tox/ci: update Python versions tested | starbase | merged | Merged into main as 201327e, updating tox and CI to test the revised set of Python versions. | |
| #332 tox: improve UX for testing | craft-providers | merged | Merged PR #332 as commit 38739ea, updating tox configuration to improve the local testing experience. | |
| #1099 ci: reference the OS to build the tox tests result filename | craft-parts | merged | Merged #1099 via commit 6200f7e8, replacing undefined matrix.platform with matrix.os when building tox result filenames. | |
| #4036 tests: ensure coverage is set per tox env | snapcraft | merged | Merged #4036 in commit ef29278e2, setting test coverage independently for each tox environment. | |
| #745 fix(ci): install tox in ci | craft-providers | merged | Merged #745 in commit 2e31f55, fixing TICS CI by installing tox, using the correct project name, and disabling parallel coverage to restore reporting. |