-
Notifications
You must be signed in to change notification settings - Fork 950
Update github actions and fix readthedocs #3983
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
Update github actions and fix readthedocs #3983
Conversation
Thanks! Maybe, let's drop the Python 3.7 workflows? |
Should we do 3.8 and 3.13 ? |
It looks like the snapshots drifted a tiny bit but CI passed. There is no bot to update the snapshots though. We can either increase the comparison threshold to make the tests more reliable or update the snapshots manually. |
I'll update the snapshots, they are in the artifacts |
…I, except for documentation test
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.
Thanks a lot for this!
The
upload-artifact@v3
anddownload-artifact@v3
actions are deprecated and prevents the workflow to run.This PR updates it to
v4
, and updates alsocheckout
action tov4
.It also fixes the documentation build by adding the sphinx configuration file in
.readthedocs.yaml
conf.