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

[WIP] refactor: use CIDv1 by default #115

Closed
wants to merge 2 commits into from

Conversation

alanshaw
Copy link
Member

@alanshaw alanshaw commented Feb 7, 2019

BREAKING CHANGE: util.cid now defaults to version 1 CIDs and created CIDs are now base32 encoded by default when stringified.

Depends on:

BREAKING CHANGE: `util.cid` now defaults to version 1 CIDs

License: MIT
Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
Copy link
Member

@vmx vmx left a comment

Choose a reason for hiding this comment

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

Could you please add a test that makes clear that it creates proper V0 CIDs if the version is passed in?

License: MIT
Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
@codecov
Copy link

codecov bot commented Feb 7, 2019

Codecov Report

Merging #115 into master will decrease coverage by 0.06%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
- Coverage   91.07%   91.01%   -0.07%     
==========================================
  Files          13       13              
  Lines         269      267       -2     
==========================================
- Hits          245      243       -2     
  Misses         24       24
Impacted Files Coverage Δ
src/util.js 97.82% <100%> (-0.1%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2ff8e1...63e9f56. Read the comment docs.

@vmx
Copy link
Member

vmx commented Jul 16, 2019

@alanshaw Is this still a thing or can it be closed?

@vmx
Copy link
Member

vmx commented Jan 13, 2020

Closing this one as I don't think it's relevant anymore.

@vmx vmx closed this Jan 13, 2020
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.

2 participants