Skip to content

Python 3.13 support: wrapping update #359

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

Closed
wants to merge 68 commits into from
Closed

Conversation

bitterpanda63
Copy link
Member

@bitterpanda63 bitterpanda63 commented Apr 14, 2025

Adds support for Python 3.13

  • Moves away from importhook and making deepcopy's to instead use wrapt
  • Adds more safety by creating @before and @after decorators which try-catch all exceptions apart from vulnerability exceptions
  • Adds more consistency by creating @on_import which also registers the packages that are wrapped
  • Adds more safety by creating patch_function, which safely patches the function, only if it exists.

Related issues: #357

…ectly' into improve-thread-caching-separated
Enhances heartbeat event validation by adding informative assertion messages, making debugging easier when validation fails.
@bitterpanda63 bitterpanda63 changed the base branch from main to improve-thread-caching-separated April 14, 2025 22:58
@bitterpanda63 bitterpanda63 self-assigned this Apr 15, 2025
@bitterpanda63 bitterpanda63 added the WIP Work In Progress label Apr 15, 2025
@bitterpanda63 bitterpanda63 changed the title Python 3.13 support Python 3.13 support: wrapping update Apr 15, 2025
@bitterpanda63 bitterpanda63 marked this pull request as ready for review April 16, 2025 12:15
Base automatically changed from improve-thread-caching-separated to main April 29, 2025 08:43
@bitterpanda63 bitterpanda63 removed the WIP Work In Progress label May 7, 2025
@bitterpanda63
Copy link
Member Author

This PR right now is too big to get merged, it's being split up into multiple PR's for reviewability

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.

2 participants