← 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 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.

Suggested action:

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
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.
qwen3.6-35b-a3b-mtp-q6 Merged a fix for the tox environment setup. Outdated references to deleted requirements files were removed from tox.ini, resolving configuration errors.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#576 build: fix tox test environments craft-parts merged Merged a one-file change fixing tox test environments in the build configuration. Approved by two reviewers with zero unresolved comments. The update adjusted three lines to resolve environment setup issues.
77%
#1755 fix(ci): fix tox ensure_version_matches charmcraft merged Merged after approval by two reviewers. Resolved a CI failure in the tox ensure_version_matches check with a single-line adjustment. The fix was successfully integrated into the codebase.
77%
#477 tox: fix broken tox environments craft-parts merged Merged a maintainer pull request fixing broken tox environments. The update modifies four files with eight additions and three deletions and received one reviewer approval before integration.
77%
#4820 fix(tox): use requirements-devel for linters snapcraft merged Merged a pull request updating the tox configuration to use requirements-devel for linters. Approved by two reviewers and passing CI, the change modifies one file to resolve linting dependencies.
76%
#1674 fix: make tox tests succeed charmcraft merged Merged to fix failing tox tests by correcting an environment variable and working around canonical/operator#1212. Approved by two reviewers with a 7-line addition and 2 deletions across three files.
74%
#310 feat: remove tox.ini starbase merged Approved by one reviewer and merged, this pull request resolves issue #309 by deleting the tox.ini file and removing 149 lines from the repository.
74%
#4036 tests: ensure coverage is set per tox env snapcraft merged Merged a single-file change (+3/-2 lines) to set test coverage per tox environment. Approved by a reviewer, passed CI, and integrated without changing coverage metrics.
73%
#66 tools: fix ruff linting command in tox starbase merged Merged a one-line fix that corrected the ruff linting command in tox. The change received reviewer approval and was successfully integrated into the codebase.
72%
#502 docs: fix tox documentation builds craft-parts merged Merged after PR #501 to fix tox documentation builds. The change updates one file with seven additions and two deletions, approved by two reviewers.
72%
#4273 docs: fix tox doc builds snapcraft merged Merged to main after reviewer approval and passing CI. The change fixes tox documentation builds by modifying a single file (+2/-12 lines), with no impact on code coverage.
72%