Skip to content

feat: remove patricia_merkle_tree dep#451

Merged
fmoletta merged 15 commits into
mainfrom
remove-patricia-trie-dep
Sep 23, 2024
Merged

feat: remove patricia_merkle_tree dep#451
fmoletta merged 15 commits into
mainfrom
remove-patricia-trie-dep

Conversation

@fmoletta

@fmoletta fmoletta commented Sep 20, 2024

Copy link
Copy Markdown
Contributor

Motivation

Remove usage of merkle_patricia_tree and replace it with our own trie implementation

Description

  • Promote modules trie & rlp to their own crate
  • Add method compute_hash_from_unsorted_iter for Trie

Closes #439

@fmoletta fmoletta changed the title feat: remove patricia_merkle_tree dip feat: remove patricia_merkle_tree dep Sep 20, 2024
@fmoletta fmoletta changed the title feat: remove patricia_merkle_tree dep feat: remove patricia_merkle_tree dep Sep 20, 2024
@fmoletta fmoletta marked this pull request as ready for review September 20, 2024 21:41
@fmoletta fmoletta requested a review from a team as a code owner September 20, 2024 21:41

@mpaulucci mpaulucci left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🚀

@fmoletta fmoletta added this pull request to the merge queue Sep 23, 2024
Merged via the queue into main with commit 8d87b85 Sep 23, 2024
@fmoletta fmoletta deleted the remove-patricia-trie-dep branch September 23, 2024 13:52
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.

Remove all reference to patricia_merkle_tree.

2 participants