-
Notifications
You must be signed in to change notification settings - Fork 91
Update infrastructure for new pythons and prep for latest sphinx #706
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
Conversation
bsipocz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is in a better shape than mine, so would merge this first and the do a bit more CI cleanup and dropping of old python.
I have a few comments that I'll commit directly, and will add back an oldest supported job, too. Once CI is all green will go ahead with the merge.
Thank you Chris!
| # Oldest known-compatible dependencies | ||
| - os: ubuntu-latest | ||
| python-version: "3.9" | ||
| sphinx: "==5.0.0" | ||
| myst-parser: "==1.0.0" | ||
| pillow: "==11.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need to keep an "oldest" job around otherwise won't really spot when we silently drop support
…rements and tox configs
| sphinx: "==5.0.0" | ||
| myst-parser: "==1.0.0" | ||
| pillow: "==11.0.0" | ||
| # Mid-range dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keeping the oldest job
| # Mid-range dependencies | |
| # Oldest known-compatible dependencies | |
| - os: ubuntu-latest | |
| python-version: "3.9" | |
| sphinx: "==5.0.0" | |
| myst-parser: "==1.0.0" | |
| pillow: "==11.0.0" | |
| # Mid-range dependencies |
|
All the test jobs are pinning sphinx <9 or fail, so I added the upper pin for now. |
This is some prep work for the new version of Sphinx, though doesn't test for it yet because a bunch of other dependencies in the stack still don't support it. So this is mostly some prep work, and the python versions shouldn't be a problem.
I think the biggest changes are: