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

Provide instructions on the PR workflow based on OWNERS files. #27

Merged
merged 3 commits into from
Mar 7, 2018

Conversation

jlewi
Copy link
Contributor

@jlewi jlewi commented Mar 3, 2018

Fix #3


This change is Reviewable

* Instructions describe how we will use prow bots and OWNERs files
to manage the PR workload.

* Instructions are based on the Kubernetes workflow
https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md#code-review-using-owners-files
@jlewi
Copy link
Contributor Author

jlewi commented Mar 6, 2018

@DjangoPeng @gaocegege @erikerlandson @ddysher Does someone want to review this?

Copy link
Member

@ScorpioCPH ScorpioCPH left a comment

Choose a reason for hiding this comment

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

@jlewi Thanks for this doc, generally LGTM, can you point out the different sections between this doc?

I find they are very similar :)

contributing.md Outdated
- [plugin: lgtm](https://git.k8s.io/test-infra/prow/plugins/lgtm)
- adds the `lgtm` label when a **reviewer** comments `/lgtm` on a PR
- the **PR author** may not `/lgtm` their own PR
- [pkg: k8s.io/test-infra/prow/repowners](https://git.k8s.io/test-infra/prow/repoowners/repoowners.go)
Copy link
Member

Choose a reason for hiding this comment

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

typo: s/repowners/repoowners

Copy link
Member

@gaocegege gaocegege left a comment

Choose a reason for hiding this comment

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

@jlewi It is awesome, LGTM

@jlewi
Copy link
Contributor Author

jlewi commented Mar 7, 2018

@ScorpioCPH Changes from the K8s doc are trivially. Main difference is "kubernetes" -> "kubeflow" and I deleted the section about mungeithub.

Don't forget to "/lgtm"

@ScorpioCPH
Copy link
Member

/lgtm

@jlewi
Copy link
Contributor Author

jlewi commented Mar 7, 2018

/approve

@gaocegege
Copy link
Member

/lgtm

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gaocegege, jlewi, ScorpioCPH

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 a485cbc into kubeflow:master Mar 7, 2018
@gaocegege
Copy link
Member

Thanks @k8s-ci-robot 😄

jlewi pushed a commit to jlewi/kubeflow that referenced this pull request Nov 26, 2019
k8s-ci-robot pushed a commit to kubeflow/kubeflow that referenced this pull request Nov 26, 2019
woop pushed a commit to woop/community that referenced this pull request Nov 16, 2020
…flow#27)

* Dynamic layout for TOC on documents.

* whitespace cleanup
saffaalvi pushed a commit to StatCan/kubeflow that referenced this pull request Feb 11, 2021
saffaalvi pushed a commit to StatCan/kubeflow that referenced this pull request Feb 12, 2021
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.

4 participants