fix(strict-deps): fail if venv is inconsistent
Metadata
Current evaluation
Merged implementation of pip check to validate charm virtualenv dependencies. The build now fails on inconsistent packages, resolving issue #1781. Approved by two reviewers and passed CI checks.
Suggested action: —
No scores available.
Issue body
This runs `pip check` to ensure that all the packages in the charm virtualenv have all of their required dependencies, failing the build if they do not.
Fixes #1781
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged implementation of pip check to validate charm virtualenv dependencies. The build now fails on inconsistent packages, resolving issue #1781. Approved by two reviewers and passed CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Adds pip check to validate charm virtualenv dependencies, failing the build on inconsistencies. Resolves #1781. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix implementing pip check to validate charm virtualenv dependencies, failing the build on inconsistencies and resolving issue #1781. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1794 fix(strict-deps): cherry-pick #1784 into 3.1 hotfix branch | charmcraft | merged | Merged cherry-pick of #1784 into the 3.1 hotfix branch. Adds pip check to validate charm virtualenv dependencies, failing the build if requirements are unmet. Resolves #1781. Approved by two reviewers with passing CI checks. | |
| #1951 feat(linter): add 'pip check' linter | charmcraft | merged | Merged a linter that executes pip check on the generated charm virtual environment to flag dependency inconsistencies. Approved by two reviewers, the update adds 263 lines across six files to enforce package integrity validation. | |
| #1891 fix(charm_builder): force-install pip | charmcraft | merged | Merged hotfix for 2.x forcing pip installation in the charm_builder venv, resolving #1456. Approved by two reviewers, passed CI, and merged successfully. |