Skip to content

ci:Update CI testing matrix for Python versions #589

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

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

bhimrazy
Copy link
Collaborator

@bhimrazy bhimrazy commented May 13, 2025

What does this PR do ?

Expand the CI testing matrix to include Python versions 3.12 and 3.13.

Copy link

codecov bot commented May 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79%. Comparing base (742fb52) to head (73e28d7).

Additional details and impacted files
@@         Coverage Diff         @@
##           main   #589   +/-   ##
===================================
- Coverage    79%    79%   -0%     
===================================
  Files        41     41           
  Lines      6143   6143           
===================================
- Hits       4847   4843    -4     
- Misses     1296   1300    +4     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@deependujha
Copy link
Collaborator

is this pr still in WIP? Is there anything else that you intend to add?

@bhimrazy
Copy link
Collaborator Author

bhimrazy commented May 17, 2025

is this pr still in WIP? Is there anything else that you intend to add?

Not really. I just had a concern: adding more Python versions might introduce unnecessary tests and increase GitHub Actions usage.

@Borda @tchaton, what are your thoughts on this? Should we expand the Python version matrix further, or is the current setup sufficient? I added for Python 3.11 last week.

Copy link
Collaborator

@tchaton tchaton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems enough

@bhimrazy bhimrazy changed the title [wip/test] ci:Update CI testing matrix for Python versions ci:Update CI testing matrix for Python versions May 23, 2025
@bhimrazy bhimrazy marked this pull request as draft May 23, 2025 06:46
@bhimrazy
Copy link
Collaborator Author

Tests on both ubuntu-22.04, 3.12 and macos-14, 3.12 appear to have fully passed (292 passed) in ~20 mins and ~30 mins respectively, but the job remains stuck in the "Tests" step and doesn't complete.

Will investigate the cause of the hang shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants