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

allow cid format selection in block put command #3324

Merged
merged 1 commit into from
Oct 30, 2016

Conversation

whyrusleeping
Copy link
Member

License: MIT
Signed-off-by: Jeromy why@ipfs.io

@whyrusleeping whyrusleeping added the status/in-progress In progress label Oct 24, 2016
@whyrusleeping whyrusleeping force-pushed the feat/block-cid-codec branch 4 times, most recently from a3f2a34 to ce3fd25 Compare October 26, 2016 00:26
@Kubuxu
Copy link
Member

Kubuxu commented Oct 28, 2016

Shouldn't it be protobuf or v0 to maintain compatibility?

@whyrusleeping
Copy link
Member Author

@Kubuxu what do you mean?

@Kubuxu
Copy link
Member

Kubuxu commented Oct 28, 2016

You had to change the hashes in the tests, which means that hashes outputted by ipfs block API have changed.

@whyrusleeping
Copy link
Member Author

Ah, yeah. Youre referring to setting the default format value to be "v0". That makes sense to me.

License: MIT
Signed-off-by: Jeromy <why@ipfs.io>
@whyrusleeping
Copy link
Member Author

@Kubuxu there, now the default behaviour doesnt change. Good catch

@whyrusleeping whyrusleeping merged commit 634005a into master Oct 30, 2016
@whyrusleeping whyrusleeping deleted the feat/block-cid-codec branch October 30, 2016 04:59
@whyrusleeping whyrusleeping removed the status/in-progress In progress label Oct 30, 2016
@ghost ghost mentioned this pull request Dec 23, 2016
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