Skip to content

Use spf13/cobra for docker pause#23285

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

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

Conversation

@yongtang
Copy link
Copy Markdown
Member

@yongtang yongtang commented Jun 5, 2016

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

Thif fix coverted command docker pause 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
@vdemeester vdemeester added the area/cli Client label Jun 6, 2016
@vdemeester vdemeester added this to the 1.12.0 milestone Jun 6, 2016
@thaJeztah
Copy link
Copy Markdown
Member

thanks, LGTM, but needs a rebase

@yongtang yongtang force-pushed the 23211-spf13-cobra-pause branch from 496c6b1 to 73149a6 Compare June 6, 2016 12:36
@yongtang
Copy link
Copy Markdown
Member Author

yongtang commented Jun 6, 2016

Thanks @cpuguy83 @thaJeztah the PR has been rebased.

@vdemeester
Copy link
Copy Markdown
Member

LGTM 🐵

@LK4D4
Copy link
Copy Markdown
Contributor

LK4D4 commented Jun 6, 2016

@yongtang need rebase :/

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

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

Signed-off-by: Yong Tang <[email protected]>
@yongtang yongtang force-pushed the 23211-spf13-cobra-pause branch from 73149a6 to 9173170 Compare June 6, 2016 19:37
@yongtang
Copy link
Copy Markdown
Member Author

yongtang commented Jun 6, 2016

Thanks. Just rebased again.

@vdemeester vdemeester merged commit 6d4a721 into moby:master Jun 7, 2016
@yongtang yongtang deleted the 23211-spf13-cobra-pause branch June 7, 2016 12:28
dnephin pushed a commit to dnephin/docker that referenced this pull request Apr 17, 2017
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.

6 participants