Skip to content

Conversation

@achingbrain
Copy link
Contributor

@achingbrain achingbrain commented Jan 9, 2023

Description

[email protected] shipped with a breaking change so update all deps using multiformats to the latest version.

BREAKING CHANGE: the CID class exposed by the types of this module now comes from [email protected]

How Has This Been Tested?

  • Unit tests pass

Definition of Done

Before submitting this PR, please make sure:

  • The work addresses the description and outcomes in the issue
  • I have added relevant tests for new or updated functionality
  • My code follows conventions, is well commented, and easy to understand
  • My code builds and tests pass without any errors or warnings
  • I have tagged the relevant reviewers
  • I have updated the READMEs of affected packages
  • I have made corresponding changes to the documentation
  • The changes have been communicated to interested parties

References:

Please list relevant documentation (e.g. tech specs, articles, related work etc.) relevant to this change, and note if the documentation has been updated.

`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.

BREAKING CHANGE: the CID class exposed by the types of this module now comes from `[email protected]`
@achingbrain achingbrain force-pushed the deps/update-multiformats-to-v11 branch from ec07523 to f1ea40a Compare January 9, 2023 10:29
@achingbrain
Copy link
Contributor Author

cc @ukstv

@achingbrain
Copy link
Contributor Author

@ukstv do you think this can go out soon? It's the last thing needed for the next js-ipfs release.

@ukstv
Copy link
Contributor

ukstv commented Jan 11, 2023

Today @achingbrain

@ukstv ukstv merged commit cd0767e into ceramicnetwork:main Jan 11, 2023
@achingbrain achingbrain deleted the deps/update-multiformats-to-v11 branch January 11, 2023 10:49
@ukstv
Copy link
Contributor

ukstv commented Jan 11, 2023

@achingbrain Released as 4.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants