Skip to content

Use spf13/cobra for docker unpause#23279

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

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

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 unpause to use spf13/cobra

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

@cpuguy83
Copy link
Copy Markdown
Member

cpuguy83 commented Jun 5, 2016

Need to remove the commented code

@yongtang
Copy link
Copy Markdown
Member Author

yongtang commented Jun 5, 2016

@cpuguy83 Sorry my bad.

Just updated the pull request. Thanks for pointing it out.

Comment thread api/client/container/unpause.go Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

think we can remove the [OPTIONS] at some point here (because we only have --help), but we can do that for all commands that don't take options in another PR

@thaJeztah
Copy link
Copy Markdown
Member

LGTM

@vdemeester vdemeester added the area/cli Client label Jun 5, 2016
@vdemeester vdemeester added this to the 1.12.0 milestone Jun 5, 2016
@yongtang yongtang force-pushed the 23211-spf13-cobra-unpause branch 2 times, most recently from 6c9eed1 to 24e605a Compare June 5, 2016 21:11
@yongtang
Copy link
Copy Markdown
Member Author

yongtang commented Jun 5, 2016

Thanks @cpuguy83 @vdemeester @thaJeztah for the review. The [OPTIONS] has been removed and the PR has been rebased to remove the conflict. Please let me know if there are any issues.

@dnephin dnephin mentioned this pull request Jun 5, 2016
43 tasks
@thaJeztah
Copy link
Copy Markdown
Member

re-LGTM

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

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

Signed-off-by: Yong Tang <[email protected]>
@yongtang yongtang force-pushed the 23211-spf13-cobra-unpause branch from 24e605a to 8ea7733 Compare June 6, 2016 00:30
@cpuguy83
Copy link
Copy Markdown
Member

cpuguy83 commented Jun 6, 2016

LGTM

@vdemeester
Copy link
Copy Markdown
Member

LGTM 🐮

@vdemeester vdemeester merged commit b70dd0e into moby:master Jun 6, 2016
@yongtang yongtang deleted the 23211-spf13-cobra-unpause branch June 6, 2016 03:26
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