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

Change IPFS to use the new pluggable Block to IPLD decoding framework. #4060

Merged
merged 1 commit into from
Jul 12, 2017

Conversation

Stebalien
Copy link
Member

Later, we should:

  1. Pull the other node formats out of IPFS (at least the raw one).
  2. Pull out the decoder registration/management into a go-ipld library.

@Stebalien
Copy link
Member Author

Replaces #4040.

Later, we should:

1. Pull the other node formats out of IPFS (at least the raw one).
2. Pull out the decoder registration/management into a `go-ipld` library.

License: MIT
Signed-off-by: Steven Allen <[email protected]>
@Stebalien Stebalien force-pushed the feat/pluggable-ipld-decoding branch from dbdaa03 to 35984c2 Compare July 12, 2017 03:35
@Stebalien Stebalien assigned Kubuxu and unassigned Kubuxu Jul 12, 2017
@Stebalien Stebalien requested a review from Kubuxu July 12, 2017 03:36
Copy link
Member

@whyrusleeping whyrusleeping left a comment

Choose a reason for hiding this comment

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

LGTM

@whyrusleeping whyrusleeping merged commit 06c567d into master Jul 12, 2017
@whyrusleeping whyrusleeping deleted the feat/pluggable-ipld-decoding branch July 12, 2017 05:13
te0d pushed a commit to te0d/go-ipfs that referenced this pull request Aug 4, 2017
Change IPFS to use the new pluggable Block to IPLD decoding framework.

cp --force: add force option to files cp command

The force option was added to the `files cp` command. Specifying force
will overwrite an existing file with the source file.

License: MIT
Signed-off-by: Tom O'Donnell <[email protected]>
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