Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use --no-build-isolation in tests to avoid using Internet #12787

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

mgorny
Copy link
Contributor

@mgorny mgorny commented Jun 22, 2024

Add --no-build-isolation to the pip invocation in pip_editable_parts fixture and a few tests, to avoid downloading setuptools wheel from PyPI. This change makes it possible again to run the vast majority of pip tests (with the exception of 3 tests) offline.

Fixes #12786.

[I don't think this is an end user-facing change, so no news item]

@ichard26 ichard26 added C: tests Testing and related things skip news Does not need a NEWS file entry (eg: trivial changes) labels Jun 22, 2024
Copy link
Member

@sbidoul sbidoul left a comment

Choose a reason for hiding this comment

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

Test failures are unrelated, but I'd rather wait until main is green before merging this.

@ichard26
Copy link
Member

Main CI is green now. Let's merge main in here and get this landed.

Add `--no-build-isolation` to the pip invocation in `pip_editable_parts`
fixture and a few tests, to avoid downloading `setuptools` wheel from
PyPI.  This change makes it possible again to run the vast majority of
pip tests (with the exception of 3 tests) offline.

Fixes pypa#12786.
@sbidoul sbidoul merged commit 1528d60 into pypa:main Jul 3, 2024
28 checks passed
@mgorny mgorny deleted the no-isol-tests branch July 3, 2024 14:16
@mgorny
Copy link
Contributor Author

mgorny commented Jul 3, 2024

Thanks!

pradyunsg pushed a commit to pradyunsg/pip that referenced this pull request Jul 7, 2024
Use `--no-build-isolation` in tests to avoid using Internet
@pradyunsg pradyunsg mentioned this pull request Jul 7, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C: tests Testing and related things skip news Does not need a NEWS file entry (eg: trivial changes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[24.1] The test suite no longer works offline
3 participants