Skip to content

Make pex accept prereleases from uv venvs.#23337

Merged
benjyw merged 1 commit into
pantsbuild:mainfrom
benjyw:allow_prereleases_from_uv_venv
May 8, 2026
Merged

Make pex accept prereleases from uv venvs.#23337
benjyw merged 1 commit into
pantsbuild:mainfrom
benjyw:allow_prereleases_from_uv_venv

Conversation

@benjyw
Copy link
Copy Markdown
Contributor

@benjyw benjyw commented May 8, 2026

When resolving with uv, it handles the resolution
decisions. If it deems it appropriate to have a
prerelease, pex should not refuse to use it.

When resolving with uv, it handles the resolution
decisions. If it deems it appropriate to have a
prerelease, pex should not refuse to use it.
@benjyw benjyw added the release-notes:not-required [CI] PR doesn't require mention in release notes label May 8, 2026
@benjyw benjyw requested review from cburroughs and sureshjoshi May 8, 2026 20:58
@benjyw benjyw added this to the 2.32.x milestone May 8, 2026
@benjyw
Copy link
Copy Markdown
Contributor Author

benjyw commented May 8, 2026

This was exposed by #23293, after merging #23284.
opentelemetry-sdk==1.41.0 introduced a dep on
prerelease opentelemetry-semantic-conventions==0.62b0
in the uv venv, that pex was refusing to use.

@benjyw benjyw merged commit 6037186 into pantsbuild:main May 8, 2026
25 checks passed
@benjyw benjyw deleted the allow_prereleases_from_uv_venv branch May 8, 2026 21:35
@WorkerPants
Copy link
Copy Markdown
Member

I tried to automatically cherry-pick this change back to each relevant milestone, so that it is available in those older releases of Pants.

✔️ 2.32.x

Successfully opened #23340.


Thanks again for your contributions!

🤖 Beep Boop here's my run link

tdyas pushed a commit that referenced this pull request May 9, 2026
…23340)

When resolving with uv, it handles the resolution
decisions. If it deems it appropriate to have a
prerelease, pex should not refuse to use it.

Co-authored-by: Benjy Weinberger <benjyw@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes:not-required [CI] PR doesn't require mention in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants