-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Pip releases should test against all versions of Python for Windows #13011
Comments
The release process creates a branch named Maybe a scheduled run of the pipeline with more tests enabled instead? |
Agreed. I think if we want to test against all Python versions, we should be doing that sufficiently before the release that we have the time to assess and fix any issues before starting the release. And in particular, anything that makes it the RM's responsibility to deal with failures here isn't reasonable - we recently changed policy to merge vendoring updates well in advance of a release in order to remove the risk that they cause issues with the release. Any testing of all Python versions should follow the same principle.
That sounds like a better idea. Maybe run it quarterly, in the month before each release is due? |
There's already a weekly scheduled test run: Lines 11 to 12 in ec5faea
Assuming there's a way to distguish that in the workflow file, seems like this would be a reasonable place to add these test runs based on feedback. |
We'd dropped this because Windows CI was exceedingly slow and we were hitting issues due to low quotas for the CI pipelines. I think we can reasonably remove that limitation now. |
IIRC we're on a [much larger] organisation plan now. Perhaps it would be easiest to run the full suite of Windows jobs across the entire Python version matrix. I'm a little worried that weekly runs may have poor visibility (AFAIK it's only maintainers that get notifications if they fail). |
What's the problem this feature will solve?
Currently for regular PRs pip only tests against the oldest supported Python version and the newest supported Python version:
pip/.github/workflows/ci.yml
Lines 175 to 182 in ec5faea
But when pip does a final release I think it would be reassuring for all versions of Python to be tested against.
Describe the solution you'd like
Somehow in the GitHub workflow file have a condition that runs all untested versions of Python for the final release.
Alternative Solutions
Do nothing.
Additional context
I don't know how to implement this yet, but if no one else does this I plan to spend a bit of time investigating this in the future and raise a PR. I'm guessing have a job associated with a specific path filter on
NEWS.rst
that tests these versions of Python on Windows.Code of Conduct
The text was updated successfully, but these errors were encountered: