Skip to content

Add errorTree type for joined, wrapped error traversal #976

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Add errorTree type for joined, wrapped error traversal #976

wants to merge 3 commits into from

Conversation

lukasschwab
Copy link
Contributor

@lukasschwab lukasschwab commented Mar 5, 2025 •
edited
Loading

Changes

  • Adds an errorTree for traversing composed errors: errors.Wrap, (github.com/pkg/errors).Wrap, and now errors.Join.
  • Refactors SetException to use errorTree.

Motivation

Fix #977

See discussion and open questions there.

@lukasschwab lukasschwab mentioned this pull request Mar 5, 2025
Open
3 tasks
@lukasschwab lukasschwab force-pushed the ls/joined-error-unwrap branch from 5eeae4f to b129962 Compare March 5, 2025 03:59
@cleptric cleptric self-requested a review March 14, 2025 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cleptric cleptric Awaiting requested review from cleptric

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Improvement Issue type
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Decompose errors.Join errors in SetException
3 participants
@lukasschwab @cleptric @stephanie-anderson