-
Notifications
You must be signed in to change notification settings - Fork 134
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 against 4.0/edge
on 22.04
#593
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #593 +/- ##
=======================================
Coverage 96.35% 96.35%
=======================================
Files 32 32
Lines 3159 3159
=======================================
Hits 3044 3044
Misses 115 115 ☔ View full report in Codecov by Sentry. |
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.
The merge-base changed after approval.
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.
Maybe remove the commits that aren't needed anymore after rebase (i.e. the ones merged through #592)?
Oh, that's probably why it didn't let me merge despite your initial ack. Indeed let me clean that up. I thought it would be smart enough to do it automatically... |
Python 3.8 was picked as it's the default python3 version on 20.04. Signed-off-by: Simon Deziel <[email protected]>
Python 3.11 makes little sense now that we test with 3.12. Also, 3.11 isn't shipped in any Ubuntu LTS version (just 23.10). Signed-off-by: Simon Deziel <[email protected]>
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.
All good!
#592 is a prerequisite
This PR is expected to fail until #590 is addressed.