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.

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

refactor: update deps cid, multihash, and ipld #309

Merged
merged 2 commits into from
Apr 30, 2024

Conversation

nathanielc
Copy link
Collaborator

@nathanielc nathanielc commented Apr 17, 2024

The rust crate libipld is deprecated and ipld-core is the new blessed ipld implementation in rust. This refactor updates our code to use this new dependency as well as up to date versions of the cid and multihash crates. There are no behavior changes in this PR.

@nathanielc nathanielc force-pushed the fix/cid-multihash-ipld branch 2 times, most recently from 08d33c5 to e6105b1 Compare April 30, 2024 15:30
@nathanielc nathanielc changed the base branch from feat/event-id-simplify to main April 30, 2024 15:30
@nathanielc nathanielc marked this pull request as ready for review April 30, 2024 15:31
The rust crate libipld is deprecated and ipld-core is the new _blessed_
ipld implementation in rust. This refactor updates our code to use this
new dependency as well as up to date versions of the cid and multihash
crates. There are no behavior changes in this PR.
Copy link
Contributor

@dav1do dav1do left a comment

Choose a reason for hiding this comment

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

One comment but LGTM. No need to see it again.

/// Unified implementation of [`recon::Store`] and [`iroh_bitswap::Store`] that can expose the
/// individual blocks from the CAR files directly.
#[derive(Clone, Debug)]
pub struct ModelStore<H>
Copy link
Contributor

Choose a reason for hiding this comment

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

This file should be safe to delete.. I moved it to sql/event.rs and it looks like you applied to changes there.

@nathanielc nathanielc added this pull request to the merge queue Apr 30, 2024
Merged via the queue into main with commit dadeea0 Apr 30, 2024
5 checks passed
@nathanielc nathanielc deleted the fix/cid-multihash-ipld branch April 30, 2024 17:58
@smrz2001 smrz2001 mentioned this pull request May 6, 2024
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