Skip to content
This repository has been archived by the owner on Nov 23, 2019. It is now read-only.

ContainerStop and ContainerRestart should use time.Duration for timeout #230

Open
aaronlehmann opened this issue May 19, 2016 · 2 comments
Open

ContainerStop and ContainerRestart should use time.Duration for timeout #230

aaronlehmann opened this issue May 19, 2016 · 2 comments
Labels
kind/enhancement

Comments

@aaronlehmann
Copy link
Contributor

These API functions take an integer number of seconds for a timeout. It would be much more idiomatic Go to use time.Duaration to specify the timeout.
@vdemeester
Copy link
Contributor

ohhh I like that 😃

@LK4D4
Copy link
Contributor

LK4D4 commented May 19, 2016

I like that too

@pdalpra pdalpra mentioned this issue May 25, 2016
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
kind/enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@vdemeester @LK4D4 @aaronlehmann