Skip to content

feat!: ProofError enum#116

Merged
Davidson-Souza merged 1 commit intomit-dci:mainfrom
Davidson-Souza:feat/proof-error
Apr 23, 2026
Merged

feat!: ProofError enum#116
Davidson-Souza merged 1 commit intomit-dci:mainfrom
Davidson-Souza:feat/proof-error

Conversation

@Davidson-Souza
Copy link
Copy Markdown
Collaborator

This commit continues our effort to improve rustreexo's error handling, by creating a struct errors generated by a proof, rather than a string.

This makes error handling easier for downstream projects, and makes the code easier to maintain. All error types that have a Proof variant have been updated with the new error type.

@Davidson-Souza Davidson-Souza marked this pull request as ready for review April 21, 2026 17:15
@luisschwab
Copy link
Copy Markdown
Contributor

just fmt

@Davidson-Souza
Copy link
Copy Markdown
Collaborator Author

just fmt

I ran pre-push, shouldn't it fix fmt issues?

This commit continues our effort to improve rustreexo's error handling,
by creating a struct errors generated by a proof, rather than a string.

This makes error handling easier for downstream projects, and makes the
code easier to maintain. All error types that have a Proof variant
have been updated with the new error type.
@luisschwab
Copy link
Copy Markdown
Contributor

Should, prepush runs fmt

@Davidson-Souza
Copy link
Copy Markdown
Collaborator Author

Should, prepush runs fmt

Nvm, I think I've changed something after running prepush.

Copy link
Copy Markdown
Contributor

@luisschwab luisschwab left a comment

Choose a reason for hiding this comment

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

ACK cd68184

@Davidson-Souza Davidson-Souza merged commit 24182e7 into mit-dci:main Apr 23, 2026
14 checks passed
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.

2 participants