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

Extract: thirdparty/tar #4857

Merged
merged 2 commits into from
Mar 23, 2018
Merged

Extract: thirdparty/tar #4857

merged 2 commits into from
Mar 23, 2018

Conversation

hsanjuan
Copy link
Contributor

This module was sort of extracted to https://github.com/whyrusleeping/tar-utils long ago.
I gxed that, imported commits from go-ipfs to the current state, and
extracted it.

License: MIT
Signed-off-by: Hector Sanjuan hector@protocol.ai

This module was sort of extracted to whyrusleeping/tar-utils long ago.
I gxed that, imported commits from go-ipfs to the current state, and
extracted it.

License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
@hsanjuan hsanjuan self-assigned this Mar 21, 2018
@hsanjuan hsanjuan requested a review from Kubuxu as a code owner March 21, 2018 17:04
@ghost ghost added the status/in-progress In progress label Mar 21, 2018
@djdv djdv mentioned this pull request Mar 21, 2018
License: MIT
Signed-off-by: Hector Sanjuan <hector@protocol.ai>
Copy link
Member

@Kubuxu Kubuxu left a comment

Choose a reason for hiding this comment

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

LGTM, it includes fix for Go1.10

@Kubuxu Kubuxu added RFM and removed status/in-progress In progress labels Mar 23, 2018
@whyrusleeping whyrusleeping merged commit 1fb90ad into master Mar 23, 2018
@whyrusleeping whyrusleeping deleted the extract/thirdparty-tar branch March 23, 2018 16:41
@whyrusleeping
Copy link
Member

Thanks @hsanjuan!

@djdv djdv mentioned this pull request Dec 31, 2018
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants