Skip to content

Conversation

gaocegege
Copy link
Member

@gaocegege gaocegege commented Apr 20, 2018

/assign @YujiOshima

/hold

I prefer to merge it after CI passes. Thus blocked by #63

Signed-off-by: Ce Gao [email protected]

Signed-off-by: Ce Gao <[email protected]>
@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: jlewi

Assign the PR to them by writing /assign @jlewi in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gaocegege
Copy link
Member Author

I am not sure if it affects frontend 🤔

@YujiOshima
Copy link
Contributor

There is nothing you should modify in the frontend files. 👍

@YujiOshima
Copy link
Contributor

LGTM. You can merge this after tests passed!

@gaocegege
Copy link
Member Author

Thanks, I am merging it since it blocks the work #46

@gaocegege gaocegege merged commit 895aaab into kubeflow:master Apr 20, 2018
@gaocegege gaocegege deleted the name branch April 20, 2018 05:50
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.

3 participants