Skip to content

Migrate exec command to cobra#23759

Merged
vdemeester merged 1 commit intomoby:masterfrom
AkihiroSuda:cobraexec
Jul 29, 2016
Merged

Migrate exec command to cobra#23759
vdemeester merged 1 commit intomoby:masterfrom
AkihiroSuda:cobraexec

Conversation

@AkihiroSuda
Copy link
Copy Markdown
Member

Part of #23211

As suggested by @vdemeester , this PR should be reviewed after release of v1.12, probably
#23211 (comment)

Cc @pdalpra @tkopczynski

@LK4D4
Copy link
Copy Markdown
Contributor

LK4D4 commented Jun 20, 2016

@AkihiroSuda fails on janky are valid

@AkihiroSuda AkihiroSuda force-pushed the cobraexec branch 2 times, most recently from c7a0b86 to de4ce75 Compare June 21, 2016 08:42
@AkihiroSuda
Copy link
Copy Markdown
Member Author

@LK4D4 done

@dnephin
Copy link
Copy Markdown
Member

dnephin commented Jun 21, 2016

LGTM

@vdemeester vdemeester added this to the 1.13.0 milestone Jun 21, 2016
@vdemeester
Copy link
Copy Markdown
Member

LGTM too 🐯
But let's wait for 1.12 release to merge 👼

@LK4D4
Copy link
Copy Markdown
Contributor

LK4D4 commented Jul 28, 2016

@AkihiroSuda It's time! pls rebase and we'll merge.

Signed-off-by: Akihiro Suda <[email protected]>
@AkihiroSuda
Copy link
Copy Markdown
Member Author

done. (change: 4f0b510)

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