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.

Sign up for GitHub

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

Jump to bottom

test: add test for dag service doing short circuit for raw.Links() #3583

Merged
merged 1 commit into from
Jan 10, 2017
Merged

test: add test for dag service doing short circuit for raw.Links() #3583

merged 1 commit into from
Jan 10, 2017

Conversation

Kubuxu
Copy link
Member

@Kubuxu Kubuxu commented Jan 10, 2017

License: MIT
Signed-off-by: Jakub Sztandera kubuxu@protonmail.ch

@Kubuxu
test: add test for dag service doing short circuit for raw.Links()
4f2bc9f 
License: MIT
Signed-off-by: Jakub Sztandera <kubuxu@protonmail.ch>
@Kubuxu Kubuxu added status/in-progress In progress need/review Needs a review status/ready Ready to be worked and removed status/in-progress In progress labels Jan 10, 2017
@Kubuxu Kubuxu added this to the ipfs 0.4.5 milestone Jan 10, 2017
@Kubuxu Kubuxu mentioned this pull request Jan 10, 2017
Closed
whyrusleeping
whyrusleeping approved these changes Jan 10, 2017
View reviewed changes
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, yay for tests only PRs

@whyrusleeping whyrusleeping mentioned this pull request Jan 10, 2017
Closed
@whyrusleeping whyrusleeping merged commit 4513732 into master Jan 10, 2017
@whyrusleeping whyrusleeping removed the status/ready Ready to be worked label Jan 10, 2017
@whyrusleeping
Copy link
Member

CHANGELOG:
Add test for raw nodes short circuiting in 'GetLinks'.

@whyrusleeping whyrusleeping deleted the test/dat-serv/raw-links-unused branch January 10, 2017 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@whyrusleeping whyrusleeping whyrusleeping approved these changes

Assignees
No one assigned
Labels
need/review Needs a review
Projects
None yet
Milestone
ipfs 0.4.5
Development

Successfully merging this pull request may close these issues.
2 participants
@Kubuxu @whyrusleeping