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

Do not allow pip compile scenario tests to discover other Python versions #1106

Merged
merged 1 commit into from
Jan 26, 2024

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Jan 25, 2024

In #1040 we broke the pip compile scenarios designed to test failure when a required Python version is not available — resolution succeeded because all of the Python versions were available in CI. Following #1105 we have the ability to isolate tests from Python versions available in the system. Here, we limit the scenarios to only the Python version in the current environment, restoring our ability to test the error messages.

With astral-sh/packse#95, we will be able to specify scenarios with access to additional system Python versions. This will allow us to include test coverage where resolution can succeed by using a version available elsewhere on the system. See #1111 for this follow-up.

@zanieb zanieb added the testing Internal testing of behavior label Jan 25, 2024
Base automatically changed from zb/bootstrap-python to main January 26, 2024 18:12
@zanieb zanieb enabled auto-merge (squash) January 26, 2024 18:13
@zanieb zanieb merged commit 91f421c into main Jan 26, 2024
5 checks passed
@zanieb zanieb deleted the zb/limit-versions branch January 26, 2024 18:18
zanieb added a commit that referenced this pull request Jan 26, 2024
…n the system (#1111)

Extends #1106 with the scenario
from astral-sh/packse#95 which tests that `pip
compile` will use the matching system Python version for builds when
available
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Internal testing of behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants