Merge pull request #527 from cdrage/update-authors-and-contributing

Update contributing doc for reviewing, update owners file
This commit is contained in:
Tomas Kral 2017-04-06 19:55:25 +02:00 committed by GitHub
commit d933317e04
2 changed files with 21 additions and 8 deletions

View File

@ -16,10 +16,12 @@ Contact one of the [OWNERS](OWNERS) on Slack to access the appropriate CLA and i
### Contributing A Patch
1. Submit an issue describing your proposed change to the repo in question.
1. The [repo owners](OWNERS) will respond to your issue promptly.
1. If your proposed change is accepted, and you haven't already done so, sign a Contributor License Agreement (see details above).
1. Fork the desired repo, develop and test your code changes.
1. Submit a pull request.
2. The [repo owners](OWNERS) will respond to your issue promptly.
3. If your proposed change is accepted, and you haven't already done so, sign a Contributor License Agreement (see details above).
4. Fork the desired repo, develop and test your code changes.
5. Submit a pull request.
Note: Code-related PR's require two ACK's / LGTM's from a maintainer while doc-related PR's require only one
### Adding dependencies

19
OWNERS
View File

@ -1,9 +1,20 @@
# See the OWNERS file documentation:
# https://github.com/kubernetes/kubernetes/blob/master/docs/devel/owners.md
# https://github.com/kubernetes/community/blob/master/contributors/devel/owners.md
assignees:
- runseb
- ngtuna
reviewers:
- cdrage
- containscafeine
- janetkuo
- kadel
- ngtuna
- runseb
- surajssd
approvers:
- cdrage
- containscafeine
- janetkuo
- kadel
- ngtuna
- runseb
- surajssd