Skip to content

Conversation

@JPHutchins
Copy link
Collaborator

No description provided.

JaagupAverin and others added 4 commits December 1, 2025 13:03
Additionally fixes an issue where `ValidationError(error)` constructor would re-raise an exception because it requires additional arguments.
@JaagupAverin
Copy link
Contributor

Looks good, thanks.
Personally I like 'raise from none' pattern to reduce error clutter, but it's up to you.

@JPHutchins
Copy link
Collaborator Author

Looks good, thanks.
Personally I like 'raise from none' pattern to reduce error clutter, but it's up to you.

That is nice, I do think it's a good fit here, feel free to change it back in the PR. Now, if only we could just remove exceptions from the language and call it a day!

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.

3 participants