-
Notifications
You must be signed in to change notification settings - Fork 52
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
dag-pb implementation of new interface? #28
Comments
Note I updated the above to include the |
|
Cool! Would it make sense to just add the above (once your PR lands and the CID dep is updated) as a new/additional export from the js-ipld-dag-pb library and call it a day? |
if we get the other changes in to |
What is the status on this work... is the legacy approach the best one here? |
https://github.com/ipld/js-dag-pb, solid and strict and has a matching Go friend now https://github.com/ipld/go-codec-dagpb |
It seems like this is sufficient to make it compatible (works in my tests locally). Is there a more "official" package already available (for compatibility reasons)?
The text was updated successfully, but these errors were encountered: