UV Pip Compile increase test coverage #2
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes:
uv_pip_compile_file_updater_spec.rb
to test everything that was tested with pip-tool'spip-compile
command. Reviewed tests one by one and either deleted them or adapted it. (See notes below)uv
to latest version. Tests need minimum 0.2.34 because it adds support for--no-binary
uv_pip_compile_options_from_compiled_file
because some options' default are different (--skip-extras, --allow-safe and --emit-build-options). Separate methods make it easier to maintain uv and pipcompile versions.Notes:
Tests removed:
—resolver legacy
which does not apply touv
. Without the legacy resolver override there is no error on pip-compile or uv pip compile.--resolver
optionsmake upgrade
. These are not relevant foruv pip compile
mode-e giturl
). As of now, uv does not support VCS Urls in editable dependencies. See Support for editable dependencies with URLs astral-sh/uv#5442Some differences/changes:
uv
outputs-e .
and pip compile-e [file://somepath/dependabot_xxxx
] (file://somepath/dependabot_xxxx%60) in the temporary file. The updated requirements.txt does not change thoughsetuptools
, unlike the current pip-compile. See more atuv pip compile
adds setuptools (and pip) to requirements.txt astral-sh/uv#1353