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

Gpe dc2153574cc69dbf519f718369eba3a609799c2f #1600

Closed
wants to merge 7 commits into from
Closed

Gpe dc2153574cc69dbf519f718369eba3a609799c2f #1600

wants to merge 7 commits into from

Conversation

rht
Copy link
Contributor

@rht rht commented Aug 23, 2015

No description provided.

@rht
Make sure ctx in commands are derived from req.Context
d32177c 
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
@rht
Replace WithTimeout with WithCancel whenever possible
5d8e150 
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
@rht
Wire a context down to (n *helpers.UnixfsNode) GetChild
34e06f6 
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
@rht
Fix 'ctx, _' to have explicit cancel
a7202fa 
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
@rht
Define discoveryConnTimeout and set it to 30s
fcf915f 
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
@rht
Replace context.TODO in test files with context.Background
30e03b2 
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
@rht
Remove ctx timeout in unixfsNode GetChild
dc21535 
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
@jbenet jbenet added the status/in-progress In progress label Aug 23, 2015
@jbenet jbenet closed this Aug 23, 2015
@jbenet jbenet removed the status/in-progress In progress label Aug 23, 2015
@rht rht deleted the gpe-dc2153574cc69dbf519f718369eba3a609799c2f branch August 23, 2015 23:54
@ajnavarro ajnavarro mentioned this pull request Aug 24, 2022
Closed
72 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rht @jbenet