Fix Python 3.11 specific RPC issue

[tovrstra]

bugfix ...

Summary by Sourcery

This pull request addresses a specific issue with RPC over sockets in Python 3.11 by adding stop events to ensure the server waits for all requests to be handled before closing. The changelog has been updated to reflect this fix, and tests that were previously skipped for Python 3.11 have been re-enabled.

• Bug Fixes:
  • Implemented a workaround for a Python 3.11 specific issue where the RPC server closes before all requests are handled by adding stop events for each handler.
• Documentation:
  • Updated the changelog to document the workaround for the Python 3.11 RPC issue.
• Tests:
  • Removed the skip conditions for tests that were previously skipped for Python 3.11 due to the RPC issue, ensuring they now run and validate the new workaround.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request addresses a specific issue with RPC handling in Python 3.11 by introducing a workaround that involves maintaining a list of stop_events to ensure all requests are handled before the server closes. The changes are documented in the changelog, and previously skipped test cases for Python 3.11 have been re-enabled.

File-Level Changes

Files	Changes
stepup/core/rpc.py docs/changelog.md	Implemented a workaround for a Python 3.11 specific issue in RPC handling and documented the change in the changelog.
tests/test_rpc.py	Re-enabled previously skipped test cases for Python 3.11 by removing the @pytest.mark.skipif decorator.

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[deepsource-io]

Here's the code health analysis summary for commits ed0e929..8a74d1e. View details on DeepSource ↗.

Analysis Summary

Analyzer	Status	Summary	Link
Shell	✅ Success		View Check ↗
Python	✅ Success		View Check ↗

---

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

[sourcery-ai]

Hey @tovrstra - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟡 General issues: 4 issues found
• 🟢 Security: all looks good
• 🟡 Testing: 1 issue found
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}