Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

feat(mpt): Migrate to thiserror#541

Merged
refcell merged 2 commits intomainfrom
cl/mpt-error-conversion
Sep 21, 2024
Merged

feat(mpt): Migrate to thiserror#541
refcell merged 2 commits intomainfrom
cl/mpt-error-conversion

Conversation

@clabby
Copy link
Contributor

@clabby clabby commented Sep 20, 2024

Overview

Migrates kona-mpt to typed errors.

@clabby clabby added K-feature Kind: feature A-mpt labels Sep 20, 2024
@clabby clabby self-assigned this Sep 20, 2024
@codecov
Copy link

codecov bot commented Sep 20, 2024

Codecov Report

Attention: Patch coverage is 74.52830% with 27 lines in your changes missing coverage. Please review.

Project coverage is 78.1%. Comparing base (72908f9) to head (b22b96b).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
crates/mpt/src/node.rs 58.3% 15 Missing ⚠️
crates/mpt/src/db/mod.rs 81.8% 4 Missing ⚠️
crates/mpt/src/errors.rs 0.0% 3 Missing ⚠️
crates/mpt/src/fetcher.rs 72.7% 3 Missing ⚠️
crates/mpt/src/list_walker.rs 89.4% 2 Missing ⚠️
Additional details and impacted files

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@refcell refcell added this pull request to the merge queue Sep 21, 2024
Merged via the queue into main with commit 9db7ced Sep 21, 2024
This was referenced Sep 21, 2024
This was referenced Oct 13, 2024
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
* feat(mpt): Migrate to `thiserror`

* rename traits
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
* feat(mpt): Migrate to `thiserror`

* rename traits
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

K-feature Kind: feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants