Skip to content

Conditionalize two new tests on the test-pypi feature#18369

Merged
charliermarsh merged 1 commit intoastral-sh:mainfrom
musicinmybrain:pypi-feature-v0.10.9
Mar 8, 2026
Merged

Conditionalize two new tests on the test-pypi feature#18369
charliermarsh merged 1 commit intoastral-sh:mainfrom
musicinmybrain:pypi-feature-v0.10.9

Conversation

@musicinmybrain
Copy link
Copy Markdown
Contributor

Summary

Both build_backend::tool_uv_build_backend_without_build_backend and build_backend::tool_uv_build_backend_wrong_build_backend fail in offline environments because they try to download a build backend from PyPI.

This PR conditionalizes both tests on the existing test-pypi feature.

Test Plan

Applied as a downstream patch to Fedora’s uv package.

Both `build_backend::tool_uv_build_backend_without_build_backend` and
`build_backend::tool_uv_build_backend_wrong_build_backend` fail in
offline environments because they try to download a build backend from
PyPI.
Copy link
Copy Markdown
Member

@charliermarsh charliermarsh left a comment

Choose a reason for hiding this comment

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

Thank you, sorry.

@charliermarsh charliermarsh merged commit 7748937 into astral-sh:main Mar 8, 2026
54 checks passed
@charliermarsh charliermarsh added the internal A refactor or improvement that is not user-facing label Mar 8, 2026
@musicinmybrain
Copy link
Copy Markdown
Contributor Author

Thank you, sorry.

Don’t worry about it! Sending a quick PR now and then is still much nicer than keeping a list of tests to skip downstream. Thanks for the swift review and merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal A refactor or improvement that is not user-facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants