Skip to content

Commit 90c6914

Browse files
shixishclaude
andcommitted
Let verify-release-test find packages on TestPyPI
uv checks --extra-index-url (prod PyPI) first and stops at the first index that has the package name. diffbot and diffbot-python both exist on prod PyPI, so the TestPyPI build was never considered. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent 4a6b0b1 commit 90c6914

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

‎Makefile‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,14 +92,16 @@ _release:
9292

9393
# Smoke-test installs from each index in a throwaway venv.
9494
# `cd $$TMP` before running python so CWD doesn't shadow the venv install with this repo's source.
95-
# Deps live on prod PyPI, so TestPyPI install needs --extra-index-url.
95+
# Deps live on prod PyPI, so TestPyPI install needs --extra-index-url. uv checks
96+
# the extra index first, and prod PyPI also has these names (diffbot only with
97+
# yanked releases), so unsafe-best-match makes it consider TestPyPI too.
9698
verify-release-test:
9799
@VERSION=$$(grep '^version' $(PYPROJECT) | head -1 | cut -d'"' -f2) && \
98100
TMP=$$(mktemp -d) && \
99101
uv venv --python 3.12 $$TMP/.venv >/dev/null 2>&1 && \
100102
uv pip install --quiet --python $$TMP/.venv/bin/python \
101103
--index-url https://test.pypi.org/simple/ \
102-
--extra-index-url https://pypi.org/simple/ \
104+
--extra-index-url https://pypi.org/simple/ --index-strategy unsafe-best-match \
103105
"$(PKG)==$$VERSION" && \
104106
(cd $$TMP && $$TMP/.venv/bin/python -c "import diffbot; print('TestPyPI install OK:', diffbot.__version__)") && \
105107
rm -rf $$TMP

0 commit comments

Comments
 (0)