← Back to issue list

running tox is broken

View original Github issue

Metadata

Project
craft-providers
Number
#84
Type
issue
State
closed
Author
jugmac00
Labels
—
Created
Updated
Closed

Current evaluation

Fixed by merged PR #86 (change 1bb3ccf, merge 8e10486), repairing tox's broken requirements.txt dependency after PR #82.

Resolution: closed resolved

Reason: PR #86, “tox: fix broken dependency on requirements.txt,” explicitly resolved #84. It was merged as 8e10486 with change commit 1bb3ccf.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This seems to be a regression from https://github.com/canonical/craft-providers/pull/82 You could avoid that local testing and CI diverges when you use `tox` also for gh actions, e.g. with the help of https://github.com/ymyzk/tox-gh-actions see e.g. https://github.com/jugmac00/flask-reuploaded/blob/master/.github/workflows/main.yml#L24-L39 and https://github.com/jugmac00/flask-reuploaded/blob/d7ec125a74065dda38b3d5623cf8f772fa70c4b8/tox.ini#L57-L63

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Fixed by merged PR #86 (change 1bb3ccf, merge 8e10486), repairing tox's broken requirements.txt dependency after PR #82.
qwen/qwen3.6-35b-a3b — closed stale Issue reported that running tox was broken due to a regression from PR #82 (unpin dependencies). The issue was closed without resolution or maintainer comment after 14 days of inactivity.

Update history

Date Change
created

Related work

  • Likely Fixed By: canonical/craft-providers#86 (confidence 100%)

    Merged fix for the broken tox dependency and the pull request is identified as resolving #84.

  • Caused By: canonical/craft-providers#82 (confidence 95%)

    #84 identified the dependency-unpinning changes in #82 as the regression source.

Related issues

Issue Project State Summary Similarity
#86 # tox: fix broken dependency on requirements.txt craft-providers merged Merged PR #86 fixed tox's broken requirements.txt dependency handling and restored unit-test execution, resolving #84.
84%
#477 tox: fix broken tox environments craft-parts merged Merged #477 fixed the broken tox environments, updating four files (+8/-3) in commit 64677494.
76%
#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.
75%
#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).
72%
#881 build(deps): update dependency tox-gh to ~=1.4.4 (main) craft-parts merged Renovate PR #881 merged into main (commit a222562e), bumping tox-gh in tox.ini from ~=1.3.2 to ~=1.4.4; 1 file changed, approved by 2 reviewers with no failing checks.
71%
#4273 docs: fix tox doc builds snapcraft merged Merged PR #4273 fixed tox documentation builds; the change landed on main as commit d25b49d63.
71%
#2178 ci: remove tox from tics charmcraft closed PR #2178 was closed unmerged after 66 days without review or comments; later #2224 (commit 484d6ed0) removed tox and freeze-requirements files.
71%
#309 Remove tox.ini starbase closed Removed obsolete tox.ini via merged PR #310 in commit 6f78e1a.
70%
#67 lint: fix ruff linting in tox craft-archives merged PR #67 was merged as commit 0148023, fixing Ruff linting configuration in tox.
70%