ci: colorize tox
Metadata
Current evaluation
Merged to enable colorized tox output in GitHub CI by overriding the TERM environment variable. Approved by two reviewers, passed all checks, and maintained 100% code coverage.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox`?
-----
Colorize the output of tox in Github CI.
Tox does not colorize its output in Github because of the envvar `TERM` is set to `dumb`. However, github handles colorized output quite nicely.
Upstream of https://github.com/snapcore/snapcraft/pull/4435 / https://github.com/canonical/craft-providers/pull/442
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to enable colorized tox output in GitHub CI by overriding the TERM environment variable. Approved by two reviewers, passed all checks, and maintained 100% code coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Overrides TERM=dumb to enable colorized tox output in GitHub CI, improving log readability without impacting coverage. Serves as an upstream dependency for related CI tooling updates. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4435 ci: colorize tox and collapse sections | snapcraft | merged | Merged into main after reviewer approval and passing CI. Integrates tox-gh to collapse CI log sections and colorizes tox output, replacing PR #4422. Minimal changes across two files. | |
| #102 tox: pass CI environment variable | starbase | merged | Merged after approval by two reviewers and passing CI checks. The change upstreams a modification to pass the CI environment variable to tox, adding two lines to one file. | |
| #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. | |
| #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. | |
| #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). | |
| #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. | |
| #1674 fix: make tox tests succeed | charmcraft | merged | Merged to fix failing tox tests by correcting an environment variable and working around canonical/operator#1212. Approved by two reviewers with a 7-line addition and 2 deletions across three files. | |
| #67 lint: fix ruff linting in tox | craft-archives | merged | Merged after reviewer approval and passing CI. Resolves ruff linting issues in the tox environment by updating three files. Code coverage remains unchanged at 88.07%. | |
| #4507 ci: update PR template to use tox commands | snapcraft | merged | Merged a two-line update to the pull request template, replacing make and pytest commands with tox. Approved by two reviewers, passed CI checks, and maintained 89.22% test coverage. | |
| #1755 fix(ci): fix tox ensure_version_matches | charmcraft | merged | Merged after approval by two reviewers. Resolved a CI failure in the tox ensure_version_matches check with a single-line adjustment. The fix was successfully integrated into the codebase. |