Thanks to visit codestin.com
Credit goes to github.com

Skip to content

v1: Fix use_effect hook with async handler#5762

Merged
FeodorFitsner merged 4 commits intomainfrom
v1-async-hook-fix
Oct 31, 2025
Merged

v1: Fix use_effect hook with async handler#5762
FeodorFitsner merged 4 commits intomainfrom
v1-async-hook-fix

Conversation

@FeodorFitsner
Copy link
Contributor

@FeodorFitsner FeodorFitsner commented Oct 31, 2025

No description provided.

Replaces generic Exception raises with more specific error types such as RuntimeError, ValueError, NotImplementedError, ImportError, and custom exceptions across multiple modules. This improves error handling clarity and aligns with Python best practices for exception usage.
Added initialization of the 'res' variable to None before executing hook setup in the Session class. This change clarifies the state of 'res' prior to its use and may help prevent potential issues with uninitialized variables.
The HTTP 413 error now includes the maximum allowed upload size in bytes for better clarity when the limit is exceeded.
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

We've reviewed this pull request using the Sourcery rules engine

@FeodorFitsner FeodorFitsner merged commit ce1d1c8 into main Oct 31, 2025
22 of 29 checks passed
@FeodorFitsner FeodorFitsner deleted the v1-async-hook-fix branch October 31, 2025 16:26
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.

1 participant