Skip to content
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

Read Python version files during toolchain installs #4335

Merged
merged 1 commit into from
Jun 17, 2024

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Jun 14, 2024

A bare uv toolchain install invocation now reads default requests from Python version files in the working directory. In order, a bare invocation means:

  • requests from .python-versions
  • a single request from.python-version
  • any installed managed toolchain
  • the latest managed toolchain download

This replaces all the functionality of cargo dev fetch-python, which we drop in #4337

@zanieb zanieb added the preview Experimental behavior label Jun 14, 2024
@zanieb zanieb force-pushed the zb/toolchain-install-many branch from 7d30506 to ab7d5ae Compare June 15, 2024 00:54
@zanieb zanieb force-pushed the zb/toolchain-install-python-version branch from de96671 to 6773f5e Compare June 15, 2024 00:54
@zanieb zanieb added the enhancement New feature or request label Jun 15, 2024
@zanieb zanieb marked this pull request as ready for review June 15, 2024 01:01
@zanieb zanieb force-pushed the zb/toolchain-install-python-version branch from 6773f5e to 411611e Compare June 16, 2024 14:47
@konstin
Copy link
Member

konstin commented Jun 17, 2024

I think we should also allow defining them in the toml configuration, something like tool.uv.python-versions

@zanieb zanieb force-pushed the zb/toolchain-install-many branch 3 times, most recently from 677b392 to 278c5c0 Compare June 17, 2024 16:32
@zanieb zanieb force-pushed the zb/toolchain-install-python-version branch from 411611e to 6c7ba3c Compare June 17, 2024 16:35
@zanieb zanieb force-pushed the zb/toolchain-install-many branch from 278c5c0 to af9eeb1 Compare June 17, 2024 16:56
@zanieb zanieb force-pushed the zb/toolchain-install-python-version branch from 6c7ba3c to fd09f43 Compare June 17, 2024 16:56
@zanieb zanieb requested a review from BurntSushi June 17, 2024 16:58
@zanieb zanieb force-pushed the zb/toolchain-install-many branch from af9eeb1 to 25427db Compare June 17, 2024 18:16
Base automatically changed from zb/toolchain-install-many to main June 17, 2024 18:24
@zanieb zanieb force-pushed the zb/toolchain-install-python-version branch from fd09f43 to 3bc3227 Compare June 17, 2024 18:28
@zanieb zanieb enabled auto-merge (squash) June 17, 2024 18:28
@zanieb
Copy link
Member Author

zanieb commented Jun 17, 2024

I think we should also allow defining them in the toml configuration, something like tool.uv.python-versions

Tracking in #4359

@zanieb zanieb merged commit 47d3834 into main Jun 17, 2024
47 checks passed
@zanieb zanieb deleted the zb/toolchain-install-python-version branch June 17, 2024 18:37
zanieb added a commit that referenced this pull request Jun 18, 2024
Adds support for reading Python version files (introduced in #4335) to
`uv venv`. If present, we'll use the file version as the default.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request preview Experimental behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants