-
Notifications
You must be signed in to change notification settings - Fork 222
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
Adding MAINTAINERS and CONTRIBUTING guidelines for DCO #53
Conversation
Signed-off-by: Jonah Kowall <[email protected]>
MAINTAINERS.md
Outdated
|
||
| Maintainer | GitHub ID | Affiliation | | ||
| --------------- | --------- | ----------- | | ||
| Placeholder | [placeholder](https://github.com/userid) | Company | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently, we have @idanl21 , @ido-opster , and @dbason as maintainers. They can get added in separate PRs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will add them in shortly for you
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Signed-off-by: Jonah Kowall <[email protected]>
You are welcome to merge now or edit, thx |
When you want to add verification opensearch-project/OpenSearch#2317 (comment) apparently there is an app for it now. |
I've installed the DCO app. Thanks for that note @jkowall |
Maintainers file is a placeholder, and Contributing file explains the guidelines around DCO which we must enforce to avoid legal issues with code contributions.
Signed-off-by: Jonah Kowall [email protected]