Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Skip test when code is not changed #467

Merged
merged 1 commit into from
Apr 24, 2019

Conversation

gaocegege
Copy link
Member

@gaocegege gaocegege commented Apr 24, 2019

Fix #374

/cc @alexandraj777

Signed-off-by: Ce Gao gaoce@caicloud.io


This change is Reviewable

Signed-off-by: Ce Gao <gaoce@caicloud.io>
@k8s-ci-robot
Copy link

@gaocegege: GitHub didn't allow me to request PR reviews from the following users: alexandraj777.

Note that only kubeflow members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

Fix #374

/cc @alexandraj777

Signed-off-by: Ce Gao gaoce@caicloud.io

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@alexandraj777
Copy link
Contributor

Thanks @gaocegege! I don't know enough about configuring prow and the structure of go projects to know if this is what we want, but it seems good. The only suggestion I'd make is to include the prow config itself...is there any way to re-run tests when the prow config file changes?

@alexandraj777
Copy link
Contributor

/assign @jlewi

@alexandraj777
Copy link
Contributor

Oh, I assigned @jlewi because it looks like he was the other person to touch these files, not sure if you're allowed to "lgtm" on your own PRs, so lmk if I've done something non-standard 😄

@hougangliu
Copy link
Member

/lgtm
/approve

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hougangliu

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

@k8s-ci-robot k8s-ci-robot merged commit 728b37b into kubeflow:master Apr 24, 2019
@gaocegege gaocegege deleted the docs branch April 25, 2019 01:33
caicloud-bot pushed a commit to caicloud/katib that referenced this pull request Apr 28, 2019
* Adding initial v1alpha2 API controller (kubeflow#457)

* Adding initial v1alpha2 controller

* Adding logs

* Adding comments

* Adding template functions for experiment

* Adding error checks

* chore: Skip test when code is not changed (kubeflow#467)

Signed-off-by: Ce Gao <gaoce@caicloud.io>

* Add serviceAccountName in UI deployment (kubeflow#469)

* feat: Introduce suggestion

Signed-off-by: Ce Gao <gaoce@caicloud.io>

* owners: Add xinyi yu

Signed-off-by: Ce Gao <gaoce@caicloud.io>
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.

Don't run full testing suite for docs/ changes
5 participants