Skip to content
This repository has been archived by the owner on Sep 28, 2021. It is now read-only.

chore: update to new multiformats #98

Merged
merged 3 commits into from
Jul 12, 2021

Conversation

achingbrain
Copy link
Member

I had to skip a couple of tests as they use the ipfs api directly which
hasn't been migrated yet so it doesn't understand the new CID class.

They can be unskipped once js-ipfs ships with the new multiformats module,
or the circular dependency between this module and js-ipfs is broken.

BREAKING CHANGE: pulls in new multiformats modules

I had to skip a couple of tests as they use the ipfs api directly which
hasn't been migrated yet so it doesn't understand the new CID class.

They can be unskipped once js-ipfs ships with the new multiformats module,
or the circular dependency between this module and js-ipfs is broken.

BREAKING CHANGE: pulls in new multiformats modules
@achingbrain achingbrain force-pushed the chore/update-to-new-multiformats branch from eba0237 to 3174b81 Compare July 10, 2021 09:41
Copy link
Member

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

LGTM

@vasco-santos vasco-santos merged commit 1641cef into master Jul 12, 2021
@vasco-santos vasco-santos deleted the chore/update-to-new-multiformats branch July 12, 2021 08:31
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.

2 participants