← Back to issue list

Fix tox environment setup

View original Github issue

Metadata

Project
craft-cli
Number
#47
Type
pull request
State
merged
Author
cjwatson
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #47 via commit 32660b2 (merge 4977287), removing stale requirements-file references from tox.ini so tox environment setup works again.

Resolution: closed resolved

Reason: PR #47 was merged as 4977287, with implementation commit 32660b2. The change removes tox.ini references to requirements files that had already been deleted, fixing environment setup.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The requirements files were removed a while ago, but `tox.ini` still referenced them.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #47 via commit 32660b2 (merge 4977287), removing stale requirements-file references from tox.ini so tox environment setup works again.
stealth/space-bunny-alpha — closed resolved Merged #47 via 32660b2, removing stale references to deleted requirements files from tox.ini so tox environments could be created correctly.
stealth/space-bunny-alpha — closed resolved tox.ini no longer references the requirements files deleted earlier, fixing environment setup in this merged, approved PR (+1/-2).
qwen/qwen3.6-35b-a3b — — Merged a fix for the tox environment setup by removing outdated references to deleted requirements files from tox.ini. Approved by two reviewers and successfully integrated.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#309 Remove tox.ini starbase closed Removed obsolete tox.ini via merged PR #310 in commit 6f78e1a.
76%
#310 feat: remove tox.ini starbase merged Merged PR #310 via commit 6f78e1a, removing the 149-line tox.ini and resolving #309.
76%
#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).
75%
#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.
75%
#477 tox: fix broken tox environments craft-parts merged Merged #477 fixed the broken tox environments, updating four files (+8/-3) in commit 64677494.
72%
#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.
72%
#199 pyclean and package tox environments starbase closed PR #310 removed tox.ini in commit 6f78e1a, making the requested pyclean and package tox-environment upstreaming obsolete; #199 was closed accordingly.
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.
70%
#332 tox: improve UX for testing craft-providers merged Merged PR #332 as commit 38739ea, updating tox configuration to improve the local testing experience.
70%