Skip to content

Migrate start command to cobra#23268

Merged
vdemeester merged 1 commit intomoby:masterfrom
WeiZhang555:cobra-start
Jun 6, 2016
Merged

Migrate start command to cobra#23268
vdemeester merged 1 commit intomoby:masterfrom
WeiZhang555:cobra-start

Conversation

@WeiZhang555
Copy link
Copy Markdown
Contributor

@WeiZhang555 WeiZhang555 commented Jun 5, 2016

Part of #23211

Signed-off-by: Zhang Wei [email protected]

ping @vdemeester, I'd like to help with the migration, hope you haven't start doing the start command,

/cc @dnephin

@WeiZhang555
Copy link
Copy Markdown
Contributor Author

WeiZhang555 commented Jun 5, 2016

@vdemeester May I also take attach, kill, restart and rm(randomly picked)?
I notice that only you are working on this, if you already start working on one of them, I'll leave it to you 😄

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

@WeiZhang555 yay cool !! For now @dnephin and I are working on this so you're help is more than welcome (and should help to make all commands into 1.12 👼)

I noted you're name (and the PR) in #23211 😉

@vdemeester vdemeester added the area/cli Client label Jun 5, 2016
@vdemeester vdemeester added this to the 1.12.0 milestone Jun 5, 2016
Comment thread api/client/utils.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.

👍

@dnephin
Copy link
Copy Markdown
Member

dnephin commented Jun 5, 2016

LGTM

@thaJeztah
Copy link
Copy Markdown
Member

Oh, sorry @WeiZhang555, this needs a rebase now

@WeiZhang555
Copy link
Copy Markdown
Contributor Author

@thaJeztah Rebased. 👍

@vdemeester
Copy link
Copy Markdown
Member

LGTM 🐮

@vdemeester vdemeester merged commit c35d9f2 into moby:master Jun 6, 2016
@WeiZhang555 WeiZhang555 deleted the cobra-start branch June 6, 2016 07:05
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.

5 participants