Skip to content

Conversation

extrange
Copy link
Contributor

@extrange extrange commented Jul 25, 2025

If --frozen is specified, then uv just uses the existing lockfile as-is, without checking pyproject.toml, defeating the purpose of the checksum calculated on pyproject.toml. If we update pyproject.toml, uv.sync.enable doesn't appear to modify the venv despite the checksum changing.

--no-install-workspace should also be removed since it is non-default uv behavior.

This makes the default experience with uv.sync.enable equal to a stock uv installation, avoiding surprises for new users.

remove --frozen and --no-install-workspace
@domenkozar
Copy link
Member

cc @lamalex

Copy link
Member

@sandydoo sandydoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems reasonable to me.

@domenkozar domenkozar merged commit e13cd53 into cachix:main Aug 5, 2025
256 of 276 checks passed
sandydoo pushed a commit that referenced this pull request Aug 21, 2025
Remove --frozen and --no-install-workspace from default uv.sync arguments
sandydoo pushed a commit that referenced this pull request Aug 21, 2025
Remove --frozen and --no-install-workspace from default uv.sync arguments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants