Skip to content
This repository was archived by the owner on Feb 12, 2024. It is now read-only.

feat: dag get cli #688

Closed
wants to merge 1 commit into from
Closed

feat: dag get cli #688

wants to merge 1 commit into from

Conversation

kumavis
Copy link
Contributor

@kumavis kumavis commented Dec 26, 2016

Here is the skeleton for the dag cli command, and an implementation of dag get <ref> where ref is { <cid> or <cid>/<path> }.

Not sure what the formatting for output should be, just logging the result as-in for now.

Was unable to run the test due to some unrelated bug on test start, but should be correct.

@jbenet jbenet added the status/in-progress In progress label Dec 26, 2016

describe('dag', () => {
describeOnlineAndOffline(repoPath, () => {
it('get', () => {
Copy link
Member

Choose a reason for hiding this comment

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

Should probably have a test-case as well when we try to resolve a hash that does not exists.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah I wanted to write more extensive tests but I couldn't even get them to run bc of some issue with a networking module

@daviddias daviddias added status/ready Ready to be worked and removed status/in-progress In progress labels Jan 19, 2017
@daviddias daviddias changed the title cli - add 'dag get' command feat: dag get cli Feb 1, 2017
@daviddias daviddias changed the base branch from feat/dag-api to master February 1, 2017 16:16
@daviddias daviddias changed the base branch from master to feat/dag-api February 1, 2017 16:16
@daviddias
Copy link
Member

@kumavis I was rebasing master on this branch to then realise this PR was made to a old branch and not against master.

Mind rebasing and PR'ing against master?

@kumavis kumavis closed this Feb 2, 2017
@Kubuxu Kubuxu removed the status/ready Ready to be worked label Feb 2, 2017
@kumavis
Copy link
Contributor Author

kumavis commented Feb 2, 2017

replaced by #753

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants