-
Notifications
You must be signed in to change notification settings - Fork 27
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
Test multiple Sphinx and OS versions #70
Conversation
These required checks will need updating: Because it can be tedious work removing and adding these in the settings when Python versions change, one option is to have a final no-op job that depends on the others. And then just mark that final as required. For example: https://github.com/hugovk/norwegianblue/actions/runs/3177642324 Would you like something similar here? |
Merging, and an issue can be opened for continue discussion. Gotta update the CI checks. Thanks! |
Fixes #36.
Includes PR #67 to avoid conflicts.
#36 (comment) says:
So let's do this, except using Python 3.7 instead of removed EOL 3.6, and testing the wheel on 3.7-3.11.
Further, #36 (comment) says:
So let's also test on Sphinx 4, 5 and 6 beta.
I did try Sphinx 2 and 3 as well but they failed due to incompatibilities of Sphinx dependencies, and it's probably not worth supporting such old versions. It would make the matrix even bigger than the current 51(!) jobs and be more complex as they don't support 3.10+ either.
The matrix could be reduced by only testing oldest and newest Python stable versions, plus RC + PyPy (i.e. 3.7, 3.10, 3.11-dev, pypy3.8)
https://github.com/hugovk/sphinxext-opengraph/actions/runs/3263561876