← Back to issue list

fix: make uv plugin re-entrant on code changes

View original Github issue

Metadata

Project
craft-parts
Number
#956
Type
pull request
State
merged
Author
bepri
Labels
—
Created
Updated
Closed

Current evaluation

Merged a fix ensuring the uv plugin detects source code changes without requiring a build space cleanup. Approved by three reviewers, passed CI, and applied with minimal changes across three files.

Suggested action: —

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ----- When changing source code, the uv plugin would not use the new code without first cleaning out the parts build space. This PR fixes that. Based off of https://github.com/canonical/craft-parts/pull/955. Needed by https://github.com/canonical/snapcraft/pull/5196. CRAFT-3841

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged a fix ensuring the uv plugin detects source code changes without requiring a build space cleanup. Approved by three reviewers, passed CI, and applied with minimal changes across three files.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix making the uv plugin re-entrant on code changes. New source code is now automatically detected and built without manual parts build space cleanup. Resolves CRAFT-3841 and enables dependent snapcraft changes.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#930 feat: add uv plugin craft-parts merged Merged the uv plugin feature, resolving issue #706 and CRAFT-2751. Approved by four reviewers, the update adds 743 lines and removes one across twelve files.
73%
#1486 docs: update uv plugin reference craft-parts merged Merged after approval. The PR updates the reference documentation by adding details for UV_COMPILE_BYTECODE and applying minor fixes to other sections. CI checks passed and changes were integrated.
73%
#107 build(deps): refresh uv.lock imagecraft merged Merged to refresh the uv.lock dependency file. Approved by two reviewers and passed CI checks. The update modified two files, adding 365 lines and removing 360.
73%
#300 fix: use uv tool run for pre-commit if not global starbase merged Merged a fix to use uv tool run for pre-commit when not installed globally. Approved by two reviewers with passing CI. Clarified that uv tracks tools independently, explaining the existing installation message.
73%
#5196 feat: support for the uv plugin snapcraft merged Merged to add uv plugin support and close issue #5187. Changes include documentation updates to index the go-use plugin and fix cross-reference warnings. Approved by four reviewers after resolving lint and test dependencies.
72%
#792 feat: add uv plugin rockcraft merged Merged the addition of the uv plugin from craft-parts to Rockcraft. Approved by three reviewers, the change modifies 31 files. Minor CI test and OSV scanner warnings were deemed unrelated and did not block integration. Closes #789.
71%
#2565 fix(plugins/uv): Avoid modifying already relocatable venv charmcraft open Avoids sed-patching the activate script for the uv plugin since craft-parts already creates relocatable venvs via uv venv --relocatable. Small 2-file diff, pending review with 0 reviewers; CI has several failing jobs (snap-tests, Trivy, typecheck-with-ty) that the author says are unrelated.
71%
#1018 build: use uv plugin to build rockcraft merged Merged to switch to the uv plugin for building. A failing CI check due to a dependency on PR #1019 was accepted, with the maintainer noting the tests will pass once the dependent PR is integrated.
70%
#1089 Backport uv fix to 2.7 craft-parts merged Merged backport of the UV 0.7 fix to the 2.7 branch for Snapcraft. Approved by two reviewers with passing CI checks on supported platforms. The change updates four files to resolve compatibility issues.
70%
#960 docs: add uv to list of plugin options and to dependency explanation rockcraft open One-line docs change adding uv to the plugin options list and dependency explanation. Approved by 2 reviewers, but 2 unresolved review comments remain and the snap-tests CI job is failing. No activity for ~11 months.
70%