Skip to content

chore: use thiserror for Error implementations#127

Merged
DaniPopes merged 2 commits into
mainfrom
dani/thiserror
Feb 1, 2026
Merged

chore: use thiserror for Error implementations#127
DaniPopes merged 2 commits into
mainfrom
dani/thiserror

Conversation

@DaniPopes
Copy link
Copy Markdown
Member

@DaniPopes DaniPopes commented Feb 1, 2026

Closes #90

@DaniPopes DaniPopes requested a review from gakonst as a code owner February 1, 2026 19:21
@DaniPopes DaniPopes enabled auto-merge (squash) February 1, 2026 19:21
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Feb 1, 2026 •
edited
Loading

Merging this PR will not alter performance

✅ 4 untouched benchmarks

Comparing dani/thiserror (a0ba24d) with main (fa56822)

Open in CodSpeed

@DaniPopes DaniPopes merged commit edc42d2 into main Feb 1, 2026
23 checks passed
@DaniPopes DaniPopes deleted the dani/thiserror branch February 1, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mattsse mattsse mattsse approved these changes

@gakonst gakonst Awaiting requested review from gakonst

@shekhirin shekhirin Awaiting requested review from shekhirin shekhirin is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Use thiserror for core::error::Error implementations

2 participants

@DaniPopes @mattsse