fix(ci): install tox in ci
Metadata
Current evaluation
Merged #745 in commit 2e31f55, fixing TICS CI by installing tox, using the correct project name, and disabling parallel coverage to restore reporting.
Resolution: closed resolved
Reason: PR #745 was merged as commit 2e31f55. The merged changes fixed all three documented TICS CI problems.
Evaluated by: mr-cal-bot
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/)?
- [ ] Have you successfully run `tox`?
-----
Includes three fixes for the TICS CI: install tox in CI, use the correct project name, don't parallelize coverage (this breaks reporting)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #745 in commit 2e31f55, fixing TICS CI by installing tox, using the correct project name, and disabling parallel coverage to restore reporting. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR fixing TICS CI by installing tox, correcting the project name, and disabling parallelized coverage to fix reporting. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #744 fix(ci): install tox in ci | craft-providers | closed | Merged PR #744 installed tox in CI for the TICS job; resolution is recorded in commit 2e31f55, which references #745. | |
| #742 fix(ci): install tox in ci for tics | craft-providers | closed | Closed unmerged; superseded by craft-providers#745, merged as commit 2e31f55 to install tox in CI and restore TICS testing. | |
| #4855 ci: enable TiCS integration | snapcraft | merged | Merged PR #4855 (69b59480e) enabled TiCS on pushes to main and added a single-result coverage tox target, resolving #4814. | |
| #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. | |
| #87 ci: configure tox to disable listing dependencies | starbase | merged | Merged PR #87 as commit e150fe2, configuring tox CI to use --no-list-dependencies and fixing #81. | |
| #295 ci: add tox | craft-providers | merged | Merged PR #295 in commit 9ebe593, enabling tox for craft-providers CI and resolving CRAFT-1726. | |
| #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. | |
| #18 ci: ensure tests succeed in CI | craft-application | merged | PR #18 was merged as commit 99ebc503, updating CI/tox configuration to ensure the test suite succeeds. | |
| #1423 build: use tox for tests | charmcraft | merged | Merged #1423 in commit 8b291de3, moving test execution to tox so tox handles Python versions while CI matrices cover operating systems. | |
| #4961 fix: use proper environment for tics | snapcraft | merged | Merged PR #4961 as commit 9501e45ad, correcting the environment used by the TiCS CI workflow. |