Skip to content
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

refactor: migrate from thiserror to snafu #174

Closed
1 task
JayWhite2357 opened this issue Sep 20, 2024 · 0 comments · Fixed by #181
Closed
1 task

refactor: migrate from thiserror to snafu #174

JayWhite2357 opened this issue Sep 20, 2024 · 0 comments · Fixed by #181
Labels
enhancement New feature or request

Comments

@JayWhite2357
Copy link
Contributor

Background and Motivation

In order to make this rep no_std compatible, we need no_std compatible errors. thiserror is not no_std compatible, while snafu is. We are currently using thiserror.

Note: While dtolnay/thiserror#304 is an open PR, it is not clear how soon this will get merged. Additionally, Substrate uses snafu, which would help unify dependencies for Substrate-based integrations.

Requested Changes

  • Replace thiserror with snafu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor!: migrate from thiserror to snafu HorizenLabs/sxt-proof-of-sql
1 participant
@JayWhite2357