ci: colorize tox and collapse sections
Metadata
Current evaluation
Merged PR #4435 (commit 77ac1119b) added tox-gh to collapse CI log sections and colorize tox output, replacing draft PR #4422. Approved by 2 reviewers.
Resolution: closed resolved
Reason: The PR itself was the resolution: it was approved by 2 reviewers and merged into main as commit 77ac1119b, updating tox config (requirements) to add tox-gh and colorize output. The only comment was an informational Codecov report.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #4435 (commit 77ac1119b) added tox-gh to collapse CI log sections and colorize tox output, replacing draft PR #4422. Approved by 2 reviewers. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #4435 (77ac1119b) added tox-gh to collapse CI log sections and colorize tox output, replacing draft PR #4422. | |
| 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. |
Update history
No update history recorded yet.
Related work
-
Duplicate Of:
snapcore/snapcraft#4422
(confidence 98%)
PR body states it replaces draft PR #4422, the earlier colourize-tox attempt
-
Related To:
canonical/craft-providers#442
(confidence 80%)
Referenced as the prior art for colorizing tox output, source of the idea
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #442 ci: colorize tox output | craft-providers | merged | Merged PR #442 as commit db8981c, overriding TERM to colorize tox output in GitHub CI; the change was also upstreamed through canonical/starbase#145. | |
| #145 ci: colorize tox | starbase | merged | Merged in canonical/starbase#145 as eae2528, overriding TERM so tox output renders in color in GitHub CI. | |
| #332 tox: improve UX for testing | craft-providers | merged | Merged PR #332 as commit 38739ea, updating tox configuration to improve the local testing experience. | |
| #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. | |
| #106 tox: improve UX | starbase | merged | Merged PR #106 as commit 8b8fed8, improving tox UX and fixing #105; it ported UX changes from craft-providers PR #332. | |
| #4422 [DRAFT] ci: improve linter output in github actions | snapcraft | closed | Closed unmerged as a draft in favor of snapcore/snapcraft#4435, merged as commit 77ac1119b to add colorized, collapsible CI output via tox-gh. | |
| #4058 tox: fix linters and makefile | snapcraft | merged | PR #4058 by lengau merged as commit be5d4307 ("tox: fix linters and makefile"), correcting linter config in tox.ini and the Makefile after the starbase tox migration (#4046). Codecov reported no coverage change. | |
| #4196 [WIP] tox CI integration improvements | snapcraft | closed | WIP PR closed unmerged with no reviews. Its tox/PR-template idea landed later as #4507 (commit f5964c9f7, 'ci: update PR template to use tox commands'). | |
| #310 feat: remove tox.ini | starbase | merged | Merged PR #310 via commit 6f78e1a, removing the 149-line tox.ini and resolving #309. | |
| #295 ci: add tox | craft-providers | merged | Merged PR #295 in commit 9ebe593, enabling tox for craft-providers CI and resolving CRAFT-1726. |