Skip to content

Use spf13/cobra for docker wait#23287

Merged
vdemeester merged 1 commit intomoby:masterfrom
yongtang:23211-spf13-cobra-wait
Jun 6, 2016
Merged

Use spf13/cobra for docker wait#23287
vdemeester merged 1 commit intomoby:masterfrom
yongtang:23211-spf13-cobra-wait

Conversation

@yongtang
Copy link
Copy Markdown
Member

@yongtang yongtang commented Jun 6, 2016

This fix is part of the effort to convert commands to spf13/cobra #23211.

Thif fix coverted command docker wait to use spf13/cobra

Signed-off-by: Yong Tang [email protected]

@cpuguy83
Copy link
Copy Markdown
Member

cpuguy83 commented Jun 6, 2016

LGTM

@yongtang yongtang mentioned this pull request Jun 6, 2016
43 tasks
@coolljt0725
Copy link
Copy Markdown
Contributor

ping @yongtang needs a rebase :)

@vdemeester vdemeester added the area/cli Client label Jun 6, 2016
@vdemeester vdemeester added this to the 1.12.0 milestone Jun 6, 2016
This fix is part of the effort to convert commands to spf13/cobra moby#23211.

Thif fix coverted command `docker wait` to use spf13/cobra

Signed-off-by: Yong Tang <[email protected]>
@yongtang yongtang force-pushed the 23211-spf13-cobra-wait branch from 7c80a3a to 82f84a6 Compare June 6, 2016 12:13
@yongtang
Copy link
Copy Markdown
Member Author

yongtang commented Jun 6, 2016

Thanks @coolljt0725. The PR has been rebased.

@vdemeester
Copy link
Copy Markdown
Member

LGTM 🐱

@vdemeester vdemeester merged commit 7a0628e into moby:master Jun 6, 2016
@yongtang yongtang deleted the 23211-spf13-cobra-wait branch June 6, 2016 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants