-
Notifications
You must be signed in to change notification settings - Fork 6
Test Python thread safety with pytest-run-parallel #42
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
Conversation
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.
Pull Request Overview
This PR updates the CI infrastructure and testing strategy by bumping GitHub Actions versions, adding parallel test execution capabilities, and dropping support for Python 3.8 and PyPy 3.8.
- Upgraded GitHub Actions (
actions/checkoutv4→v5,actions/setup-pythonv5→v6) - Added pytest-run-parallel for running tests multiple times in parallel
- Removed Python 3.8 and PyPy 3.8 from the test matrix
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| dev_requirements.txt | Added pytest-run-parallel dependency for parallel test execution |
| .github/workflows/integration.yaml | Updated GitHub Actions versions, removed Python 3.8 support, and enabled parallel test execution with 8 iterations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
5afde51 to
21215b0
Compare
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.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
21215b0 to
ce5ec55
Compare
507dc1e to
7b3c5cc
Compare
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.
Pull Request Overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Christian Clauss <cclauss@me.com>
7b3c5cc to
43f1d9a
Compare
|
Closing due to lack of support for free threaded Python in this module at this moment. See #52 (comment) |
https://py-free-threading.github.io
https://pypi.org/project/pytest-run-parallel
%
uvx --with="valkey[libvalkey]" python3.14t -c "import valkey"