Skip to content
This repository has been archived by the owner on Aug 12, 2020. It is now read-only.

fix: updates ipld-dag-pb dep to version without .cid properties #233

Merged
merged 1 commit into from
Nov 12, 2018

Conversation

achingbrain
Copy link
Collaborator

@achingbrain achingbrain commented Nov 7, 2018

Follows on from ipld/js-ipld-dag-pb#99 and updates this module to not rely on DAGNodes having knowledge of their CIDs.

Bonus: also removes use of js-ipfs from this module breaking another circular dependency from the project.

@ghost ghost assigned achingbrain Nov 7, 2018
@ghost ghost added the status/in-progress In progress label Nov 7, 2018
@achingbrain
Copy link
Collaborator Author

N.b. this PR uses a github url for the ipld-dag-pb dependency temporarily

@achingbrain achingbrain force-pushed the update-dag-pb-to-not-have-cid-property branch 3 times, most recently from 0b97527 to ab4ef03 Compare November 8, 2018 10:29
@achingbrain achingbrain force-pushed the update-dag-pb-to-not-have-cid-property branch 2 times, most recently from 9cf2223 to 4aaae9a Compare November 11, 2018 09:02

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Follows on from ipld/js-ipld-dag-pb#99 and
updates this module to not rely on DAGNodes having knowledge of
their CIDs.

Bonus: also removes use of js-ipfs from this module breaking another
circular dependency from the project.
@achingbrain achingbrain force-pushed the update-dag-pb-to-not-have-cid-property branch from 4aaae9a to aa61cce Compare November 12, 2018 07:35
@achingbrain achingbrain merged commit dc67d6f into master Nov 12, 2018
@achingbrain achingbrain deleted the update-dag-pb-to-not-have-cid-property branch November 12, 2018 09:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant