-
Notifications
You must be signed in to change notification settings - Fork 82
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
Add testing for Python 3.12 #859
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #859 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 13 13
Lines 721 721
=========================================
Hits 721 721 ☔ View full report in Codecov by Sentry. |
Will do it! |
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.
bors merge
@sanders41 looks like there are merge conflict and so I cannot merge the branch |
@curquiza I merged main and it didn't show me any conflicts...we will see if the merge is enough |
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.
Done ✅ |
Pull Request
Python 3.12 is scheduled for release today so this PR adds 3.12 to the test matrix. The release has not been completed yet so tests will fail until that happens. I have tested locally with the RC and all tests pass.
We also need to add 3.12 to the required tests in the GitHub settings once the release happens and this comes out of draft.
Related issue
Fixes #<issue_number>
What does this PR do?
PR checklist
Please check if your PR fulfills the following requirements:
Thank you so much for contributing to Meilisearch!