ci: colorize tox and collapse sections
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `pytest tests/unit`?
-----
- Add [`tox-gh`](https://github.com/tox-dev/tox-gh) from starbase, which adds collapsing sections in CI logs
- Colorize the output of tox (similar to https://github.com/canonical/craft-providers/pull/442)
Replaces https://github.com/snapcore/snapcraft/pull/4422
[Before](https://github.com/snapcore/snapcraft/actions/runs/6721498228/job/18267360658#step:5:1)
[After](https://github.com/snapcore/snapcraft/actions/runs/6734011448/job/18303944002?pr=4435#step:5:1)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Adds tox-gh for collapsible CI log sections and colorized tox output, replacing PR #4422. Codecov approved the merge despite a minor coverage decrease. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Integrated tox-gh to collapse CI log sections and colorized tox output, replacing PR #4422. Codecov reported a 0.03% coverage decrease with 90.16% diff coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #145 ci: colorize tox | starbase | merged | 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. | |
| #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. | |
| #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). | |
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #502 docs: fix tox documentation builds | craft-parts | merged | Merged after PR #501 to fix tox documentation builds. The change updates one file with seven additions and two deletions, approved by two reviewers. | |
| #44 docs: Tox environments and labels | starbase | merged | Merged documentation update detailing Tox environments and labels. Approved by one reviewer with zero unresolved comments. The change adds 20 lines to a single file. |