-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
How can I integrate tox
into uv
?
#1484
Comments
https://tox.wiki/en/latest/config.html#install_command looks to be a possible solution to putting uv in place of pip |
No, It does not allow as uv is not installed in venv, and you cannot install it when you change install command... To help myself with this task, I started implementing tox plugin. https://github.com/Czaki/tox-uv But I hit the bug in
|
@Czaki -- We can install via a full path, we just require a package name right now (like |
@charliermarsh Ok. I do not control this part of tox, and it will need to do hacking that may not be stable. Why provide a full path to sdist or wheel is not enough and require extracting it from the file name? |
It's just not implemented yet. In general, we can't read the package name from sdist files (like the one you linked -- that looks like an sdist?) since it doesn't follow any particular spec. Wheels do follow a spec and so we could read the package name from the filename in that case. |
|
Ok. I have created a workaround. It works nice with wheels, but end with error for sdist:
|
Related to #313 |
tox devs published owns tox-uv plugin https://github.com/tox-dev/tox-uv (but it crashes tox in my case) |
@charliermarsh now that https://pypi.org/project/tox-uv/ is launched, I think we can close this. Unless you want to document it somewhere, the advice to use that. sdists seem to still be troublesome occasionally, but otherwise work well. |
Closing as |
I find that
tox
can take a long time to install dependencies (longer than doing it locally), it would be nice to benefit fromuv
for this. Think that the relevant bit in tox is here.The text was updated successfully, but these errors were encountered: