Skip to content

Add tox to lockfile - #1608

Merged
hynek merged 2 commits into
mainfrom
lock-tox
Aug 3, 2026
Merged

Add tox to lockfile#1608
hynek merged 2 commits into
mainfrom
lock-tox

Conversation

@hynek

@hynek hynek commented Aug 3, 2026

Copy link
Copy Markdown
Member

Better reproducibility everywhere.

Comment thread tox.ini
requires =
tox>=4.51.0
tox-uv
tox-uv-bare

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, what? Is it published under two names now? 🤔

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bare doesn’t install uv as a dependency

Comment thread tox.ini
@@ -3,7 +3,7 @@
[tox]
requires =

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a use case for having tox to implement referencing this section from dep groups to dedupe the list..

cc @gaborbernat

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Im This Vase it’s more of. Belts and suspenders thing since we always run tox using uv. Maybe should be removed to fail loudly? Idk

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, my understanding is that this is for people that have tox as a primary pre-existing entry-point (and not uv) — tox then makes a managed venv under .tox/.tox/ and installs all the goodies + a newer version of itself. So to me this sounds like a more logical hook point for sticking automatic provisioning. I'm wondering, though, if tox's API allows plugins to extend this list — this would make for a powerful mechanism of setting up centralized ecosystem-centric infras.

I think this might also be in use by downstreams. But we'd have to ask @hroncok @befeleme @encukou @mgorny etc.

@hynek
hynek merged commit 65c80d7 into main Aug 3, 2026
20 of 21 checks passed
@hynek
hynek deleted the lock-tox branch August 3, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants