-
Notifications
You must be signed in to change notification settings - Fork 431
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
Use Ubuntu 22.04 for CI #1859
Use Ubuntu 22.04 for CI #1859
Conversation
Thew new package name for |
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.
LGTM!
56 unit tests are being skipped. |
But they are currently skipped everywhere, also on master.
Thanks, added back. |
The tests that are skipped are from
The gramps60 branch is not yet available for the addons, so this is understandable. These are also the tests that require the lxml functionality. |
I'm confused, nothing added in this PR changes the number of tests skipped compared to master, right? |
A change in OS configuration? gramps/gramps/gen/merge/test/merge_ref_test.py Lines 44 to 45 in ca6205d
|
We are trying to load the v5.2 version of CliMerge and ExportRaw which is going to fail. The tests are then skipped. Everything is working in the gramps52 branch. I checked the unit tests as part of the v5.2.4 release process. |
Rebased. |
In #1858, I realized the CI currently runs on Ubuntu 20.04, which uses Python 3.8, that we are not supporting anymore. This PR switches conservatively to 22.04 LTS, which uses Python 3.10. One package had to be removed as well to make it run, but I believe this is anyway an old Python 2 leftover.