tox-dev / tox-uv

Use https://github.com/astral-sh/uv with tox
MIT License
112 stars 17 forks source link

Bump uv version #36

Closed hkad98 closed 7 months ago

gaborbernat commented 7 months ago

Why?

hkad98 commented 7 months ago

@gaborbernat 0.1.17 presents significant speed up.

See: https://x.com/charliermarsh/status/1767251713134653641?s=46&t=ZI8e2h9kRvR6J2B6m0-I-Q

Stealthii commented 7 months ago

Minimum versions are constraints, ideally defining the lowest version providing support for the features used within a package (tox-uv, see #28).

Unless tox-uv makes exclusive use of new features provided in these newer versions (which aren't included in this PR), this isn't a good PR, only restricting further what versions of the tool are supported.

hkad98 commented 7 months ago

@Stealthii you are right. This was just a quick PR to use the latest supported uv which can bring significant improvement in performance.

gaborbernat commented 7 months ago

Thing is you don't need this PR for that, if you reinstall the tool it will automatically pull it in.

hkad98 commented 7 months ago

@gaborbernat what is the purpose of uv dependency restriction in pyproject.toml then?

gaborbernat commented 7 months ago

it's lower restricted, meaning we need at least this version.