Skip to content
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

chore: update multicodec #210

Closed
wants to merge 2 commits into from
Closed

chore: update multicodec #210

wants to merge 2 commits into from

Conversation

hacdias
Copy link
Member

@hacdias hacdias commented Dec 23, 2019

Updating multicodec to 1.0.0 as we decided to bump it to 1.0.0 and deprecate 0.x.

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
@adamsoffer
Copy link

adamsoffer commented Dec 23, 2019

@hacdias this commit seems to have caused a compilation error in applications that depend on this library.

image

I believe this is because it's referring to a file that's been removed in the latest multicodec release (name-table.js)?

@adamsoffer
Copy link

ah looks like this was addressed here #209

@hacdias hacdias requested a review from alanshaw December 24, 2019 08:50
@hacdias
Copy link
Member Author

hacdias commented Dec 24, 2019

@adamsoffer yes, and only Windows is failing, as on master. Merry Christmas too!

@achingbrain
Copy link
Member

This was done as part of #206

@achingbrain achingbrain deleted the chore/update-multicodec branch May 27, 2020 15:10
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.

3 participants