Switch the pants repo to use uv lockfiles#23293
Conversation
7e40d73 to
3f40bdc
Compare
52fa9d0 to
d9e88e2
Compare
d9e88e2 to
46cd6bb
Compare
|
Reviewers: The first commit includes the very minor manual changes and can use human scrutiny. The second is the very large number of changes created by |
|
We didn't lose anything noteworthy in those 10k lines? |
|
Also, did this come with any version updates on that generation? |
No, the info in the files is equivalent (pex lockfiles have a few more lines of metadata, but that's not all that significant). It's more that the uv format crams more info per line. |
It did. I should probably mention those in the release notes. |
|
Added to the release notes. |
|
I think dogfooding with I'm less sure about switching all of the bundled tools lockfiles. That seems like a broader change. Do we know for example if this is faster or slower when used? |
I don't think we're cherry picking this into 2.32, so I specifically think we SHOULD try this now and revert it later if things go off the rails. |
|
Definitely not for cherry-picking. |
678a7f2 to
e63ed0e
Compare
|
Noting that this failed - spectacularly - on main due to issues users have already pointed out, but were seemingly ignored. The current treatment does not work with remote caching: #23344 (comment) |
|
Yikes, that is bad (of me). Revert here: #23360 I did not remember that we use remote caching (only on intra-fork merges). |
Includes in-repo lockfiles and the ones we
embed as default lockfiles for Python tools.
Does not include a handful of lockfiles used
in tests. Those will be done in a followup.