Skip to content
This repository has been archived by the owner on Feb 1, 2023. It is now read-only.

feat: add a custom CID type #308

Merged
merged 1 commit into from
Mar 19, 2020
Merged

feat: add a custom CID type #308

merged 1 commit into from
Mar 19, 2020

Conversation

Stebalien
Copy link
Member

This allows us to marshal/unmarshal/size protobufs without copying CID around.

@Stebalien Stebalien mentioned this pull request Mar 18, 2020
Copy link
Contributor

@dirkmc dirkmc left a comment

Choose a reason for hiding this comment

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

Nice! 👍

This should be backwards compatible right?

@Stebalien
Copy link
Member Author

This should be backwards compatible right?

Should be. But let me add a quick test.

This allows us to marshal/unmarshal/size protobufs without copying CID around.
@Stebalien
Copy link
Member Author

Ok, I've added a test with a known-good message with a CID.

@Stebalien Stebalien merged commit 21d8834 into master Mar 19, 2020
@Stebalien Stebalien deleted the feat/custom-cid-type branch March 19, 2020 01:24
@dirkmc
Copy link
Contributor

dirkmc commented Mar 19, 2020

Awesome thank you 👍

Jorropo pushed a commit to Jorropo/go-libipfs that referenced this pull request Jan 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants