Skip to content

Conversation

Jeehut
Copy link
Member

@Jeehut Jeehut commented Dec 24, 2024

Fixes #18.

You might want to review #14 first, this builds upon it (but the "Files changed" tab doesn't contain any changes from the other PR because I targeted this towards that branch so you only see the changes of this PR).

@Jeehut Jeehut requested a review from n0rthk1n9 December 24, 2024 09:11
Base automatically changed from wip/common-errors to main January 14, 2025 10:52
Copy link
Collaborator

@n0rthk1n9 n0rthk1n9 left a comment

Choose a reason for hiding this comment

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

Pretty clever way of reducing boilerplate and simplifying tedious error handling! I like it. This looks good to me, can be merged.

@Jeehut Jeehut merged commit 6dc521d into main Jan 19, 2025
1 check passed
@Jeehut Jeehut deleted the wip/nested-errors branch January 19, 2025 09:09
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.

ErrorKit: Built-in nestable generic error type

2 participants