Skip to content

Install the same uv version in docker as used by uv2nix #1037

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

juliankuners
Copy link
Contributor

@juliankuners juliankuners commented Jun 3, 2025

Some time ago, we migrated from poetry to uv. At first, the version of uv was pinned in CI in #1023. Later, in #1028, the CI workflows were adapted to install uv using the nix flake to match the version used by CI with the version used by the nix derivation. Though this pull request did not include updates to the docker image, where the version of uv that is installed is still fixed.

This pull request adapts the Dockerfile and Docker GitHub action to install the same version of uv as is used by the nix derivation. It does so by querying the uv version from deps/uv_release. deps/uv_release in turn is updated by update-version.yml.

It does so by querying the uv version from the uv2nix GitHub repository and validating that it is indeed a version string.

Additionally, this pull request adapts the GitHub workflows to also install uv using this method instead of nix to make the used method consistent everywhere. It also makes the used version somewhat more explicit.

@juliankuners juliankuners force-pushed the uv2nix-uv-version-in-dockerfile branch 2 times, most recently from f56fe5b to 020e341 Compare June 3, 2025 11:36
@juliankuners juliankuners force-pushed the uv2nix-uv-version-in-dockerfile branch from 020e341 to a3a1b2e Compare June 3, 2025 11:38
@juliankuners juliankuners marked this pull request as draft June 3, 2025 14:56
@juliankuners juliankuners marked this pull request as ready for review June 3, 2025 18:41
@automergerpr-permission-manager automergerpr-permission-manager bot merged commit c240a35 into master Jun 24, 2025
9 checks passed
@automergerpr-permission-manager automergerpr-permission-manager bot deleted the uv2nix-uv-version-in-dockerfile branch June 24, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants