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

Remove dependency on IPFS #24

Closed
achingbrain opened this issue Nov 6, 2018 · 0 comments · Fixed by #25
Closed

Remove dependency on IPFS #24

achingbrain opened this issue Nov 6, 2018 · 0 comments · Fixed by #25
Assignees

Comments

@achingbrain
Copy link
Collaborator

We pass in and instance of js-ipfs to all core modules in this repo but all MFS operations could just use js-ipld and js-interface-datastore directly instead.

We should refactor the core to accept these as arguments instead and so remove the circular dependency of js-ipfs -> js-ipfs-mfs -> js-ipfs.

achingbrain added a commit that referenced this issue Nov 8, 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 fixes #24 by removing use of js-ipfs from this module
breaking another circular dependency from the project.

License: MIT
Signed-off-by: achingbrain <[email protected]>
@ghost ghost assigned achingbrain Nov 8, 2018
@ghost ghost added the in progress label Nov 8, 2018
achingbrain added a commit that referenced this issue Nov 8, 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 fixes #24 by removing use of js-ipfs from this module
breaking another circular dependency from the project.

License: MIT
Signed-off-by: achingbrain <[email protected]>
achingbrain added a commit that referenced this issue Nov 8, 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 fixes #24 by removing use of js-ipfs from this module
breaking another circular dependency from the project.

License: MIT
Signed-off-by: achingbrain <[email protected]>
achingbrain added a commit that referenced this issue Nov 8, 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 fixes #24 by removing use of js-ipfs from this module
breaking another circular dependency from the project.

License: MIT
Signed-off-by: achingbrain <[email protected]>
achingbrain added a commit that referenced this issue Nov 8, 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 fixes #24 by removing use of js-ipfs from this module
breaking another circular dependency from the project.

License: MIT
Signed-off-by: achingbrain <[email protected]>
achingbrain added a commit that referenced this issue Nov 8, 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 fixes #24 by removing use of js-ipfs from this module
breaking another circular dependency from the project.

License: MIT
Signed-off-by: achingbrain <[email protected]>
achingbrain added a commit that referenced this issue Nov 12, 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 fixes #24 by removing use of js-ipfs from this module
breaking another circular dependency from the project.

License: MIT
Signed-off-by: achingbrain <[email protected]>
@ghost ghost removed the in progress label Nov 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
1 participant