Closed pjdon closed 6 months ago
Thanks, this makes sense. I just tweaked the code a bit to respect the RYE_TOOLCHAIN_VERSION
if it is set. I could see an argument either way... but I think the set env var overriding a prompted "default" version is reasonable.
Fixes https://github.com/astral-sh/rye/issues/1024
Currently:
ensure_self_venv_with_toolchain
to bootstrap rye internals it would pass thetoolchain_version_request
variable as the version to installRYE_TOOLCHAIN_VERSION
env var is not set, then the value oftoolchain_version_request
remains Noneprompt_for_default_toolchain
is called to prompt the user for a version to select, asking the user"Which version of Python should be used as default toolchain?"
config_doc
TOML configuration and returns nothingChanges:
toolchain_version_request
mutable so it can be updated by the user's inputprompt_for_default_toolchain
returns the resolved version that the user wants, which may be the input they typed or the default valuetoolchain_version_request
so it can be used during the internals bootstrapping