← Back to issue list

build(deps): constrain setuptools-scm due to cyclic dependency

View original Github issue

Metadata

Project
charmcraft
Number
#2602
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged to constrain setuptools-scm and resolve a cyclic dependency. Approved by two reviewers, it implements a workaround for upstream issue #1302 by pinning the package version. CI checks passed before integration.

Suggested action:

No scores available.

Issue body

Workaround for https://github.com/pypa/setuptools-scm/issues/1302 Snap build not tested yet, but it works with `UV_PYTHON=3.12 UV_PYTHON_DOWNLOADS=never UV_NO_BINARY=1 uv sync --frozen --no-dev --no-editable --no-cache` Describe your changes. --- - [ ] I've followed the [contribution guidelines](https://github.com/canonical/charmcraft/blob/main/CONTRIBUTING.md). - [ ] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [ ] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged to constrain setuptools-scm and resolve a cyclic dependency. Approved by two reviewers, it implements a workaround for upstream issue #1302 by pinning the package version. CI checks passed before integration.
qwen3.6-35b-a3b-mtp-q6 Merged a constraint for setuptools-scm to resolve a cyclic dependency issue. The workaround was successfully integrated despite non-blocking type-checking notifications.
qwen3.6-35b-a3b-mtp-q6 Merged to constrain setuptools-scm and resolve a cyclic dependency. A type-checking bot warning was informational only. The change was successfully integrated into the codebase.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#306 build(deps): constrain setuptools-scm due to cyclic dependency imagecraft merged Merged a dependency update constraining setuptools-scm to resolve a cyclic dependency. Approved by two reviewers and passed most CI checks before being merged.
92%
#6123 build(deps): constrain setuptools-scm due to cyclic dependency snapcraft merged Merged to constrain setuptools-scm and resolve a cyclic dependency. Approved by two reviewers, passed CI, and applied with a six-line addition across two files.
91%
#1158 build(deps): constrain setuptools-scm due to cyclic dependency rockcraft merged Merged a minimal dependency constraint to resolve a cyclic setuptools-scm issue. Approved by a reviewer and validated by CI, the change successfully implements a workaround for the upstream bug.
91%
#105 build(deps): constrain setuptools-scm due to cyclic dependency debcraft merged Merged to constrain setuptools-scm and resolve a cyclic dependency. Approved by two reviewers and passed most CI checks despite one failing OSV-scanner. The change updates two files, adding 10 lines and removing 430.
89%
#110 build(deps): unpin setuptools-scm debcraft merged Merged unpinning of the setuptools-scm dependency after the upstream bug was resolved. Approved by two reviewers and passed CI checks, updating version constraints across two files.
86%
#309 build(deps): unpin setuptools-scm imagecraft merged Merged after approval, unpinned setuptools-scm dependency following upstream bug resolution. CI passed and version constraints were removed across two files.
82%
#2615 build(deps): unpin setuptools-scm charmcraft merged Merged unpinning of setuptools-scm after upstream bug resolution. Approved by two reviewers, the change modified two files with a net reduction of five lines. CI passed despite non-blocking type warnings, successfully removing the dependency constraint.
81%
#1164 build(deps): unpin setuptools-scm rockcraft closed Merged after the upstream bug was fixed. The setuptools-scm dependency pin was removed, and all CI checks passed.
78%
#1157 build: constrain setuptools-scm rockcraft merged Merged a PR constraining the setuptools-scm dependency to resolve Pydantic build hangs caused by version 10.0.2. Approved by reviewers and passing CI, the change updates the build configuration with minimal modifications.
78%
#6131 build(deps): unpin setuptools-scm snapcraft merged Merged the unpinning of setuptools-scm following the upstream resolution in version 10.0.5. The pull request received two approvals, passed CI checks, and removed the version constraint across two files.
78%