Skip to content
This repository has been archived by the owner on Apr 29, 2020. It is now read-only.

Added Group tar #53

Merged
merged 3 commits into from
Mar 2, 2016
Merged

Added Group tar #53

merged 3 commits into from
Mar 2, 2016

Conversation

RichardLitt
Copy link
Contributor

No description provided.

'ipfs tar add' will parse a tar file and create a merkledag structure to represent it.

+ Parameters
+ arg (string, required) - tar file to add
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this the file itself or the path to the tar file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The file itself. How can I make that clearer?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure :/ cc @diasdavid

@RichardLitt
Copy link
Contributor Author

@diasdavid or @dignifiedquire: review?

@dignifiedquire
Copy link
Collaborator

LGTM

+ Headers

```
Access-Control-Allow-Headers: X-Stream-Output, X-Chunked-Output
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, no ouptut anymore, I've noticed that in go-ipfs too :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ahahaha I never noticed that.

@daviddias
Copy link
Contributor

LGTM overall :)

@daviddias daviddias removed their assignment Mar 2, 2016
RichardLitt added a commit that referenced this pull request Mar 2, 2016
@RichardLitt RichardLitt merged commit faba217 into master Mar 2, 2016
@RichardLitt RichardLitt deleted the feature/tar branch March 2, 2016 16:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants