# tox: fix broken dependency on requirements.txt
Metadata
Current evaluation
Merged PR #86 fixed tox's broken requirements.txt dependency handling and restored unit-test execution, resolving #84.
Resolution: closed resolved
Reason: PR #86 was merged in commit 8e10486, with implementation commit 1bb3ccf. It corrected the tox dependency regression introduced when PR #82 unpinned dependencies.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This is a more complex project, so to start simple and ensure things
keep working with tox the first item to be replaced are the unit
tests.
Fixes #84
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
-------
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #86 fixed tox's broken requirements.txt dependency handling and restored unit-test execution, resolving #84. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR fixing broken tox dependency on requirements.txt, resolving #84. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
craft-providers#84
(confidence 100%)
PR #86 explicitly fixes the broken tox execution reported in issue #84.
-
Caused By:
craft-providers#82
(confidence 98%)
Issue #84 identified PR #82's dependency-unpinning change as the regression trigger.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #84 running tox is broken | craft-providers | closed | Fixed by merged PR #86 (change 1bb3ccf, merge 8e10486), repairing tox's broken requirements.txt dependency after PR #82. | |
| #47 Fix tox environment setup | craft-cli | merged | Merged PR #47 via commit 32660b2 (merge 4977287), removing stale requirements-file references from tox.ini so tox environment setup works again. | |
| #4820 fix(tox): use requirements-devel for linters | snapcraft | merged | PR #4820 merged as commit e6a0f5c3a, adding 4 lines to tox.ini so the lint environment installs linter dependencies from requirements-devel instead of an ad-hoc list. tox.ini was later deleted (18a7596c9). | |
| #477 tox: fix broken tox environments | craft-parts | merged | Merged #477 fixed the broken tox environments, updating four files (+8/-3) in commit 64677494. | |
| #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. | |
| #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. |