Repository navigation
fix: ambigious tilde specifier requires-python with--meta=uv - #1321
dbanty merged 1 commit into
Conversation
Sorry, something went wrong.
dbanty
left a comment
There was a problem hiding this comment.
Thanks!
Sorry, something went wrong.
|
That was fast, thanks @dbanty! |
Sorry, something went wrong.
> [!IMPORTANT] > Merging this pull request will create this release ## Fixes - ambigious tilde specifier requires-python with`--meta=uv` (#1321) Co-authored-by: knope-bot[bot] <152252888+knope-bot[bot]@users.noreply.github.com>
| description = "A client library for accessing Test 3.1 Features" | ||
| authors = [] | ||
| requires-python = "~=3.9" | ||
| requires-python = ">=3.9,<4.0" |
There was a problem hiding this comment.
When evaluating requires-python ranges for dependencies, uv only considers lower bounds and ignores upper bounds entirely. For example, >=3.8, <4 is treated as >=3.8
I think we should consider dropping the upper bound from the template due to the various issues it's caused and that some tools ignore it anyway.
This might be a holdover from when this package was more Poetry-focused and Poetry used to by default put a <4 upper bound on new projects created with it? They've since stopped doing that, so I think we should consider doing the same.
Sorry, something went wrong.
There was a problem hiding this comment.
Fine by me, seems like there will never be a Python 4 anyway 😋
Sorry, something went wrong.
There was a problem hiding this comment.
Sorry, something went wrong.
Clients generated like this:
Generate these warnings during installation with uv: