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

Introduce MAINTAINERS.md #662

Merged
merged 2 commits into from
Mar 11, 2020
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions MAINTAINERS.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# Kubernetes Event-driven Autoscaling (KEDA) Maintainers

## Maintainers

| Maintainer | GitHub ID | Affiliation |
| -------------------- | --------------------------------------------- | ----------- |
| Jeff Hollan | [jeffhollan](https://github.com/jeffhollan) | Microsoft |
| Anirudh Garg | [anirudhgarg](https://github.com/anirudhgarg) | Microsoft |
| Aarthi Saravanakumar | [Aarthisk](https://github.com/Aarthisk) | Microsoft |
| Yaron Schneider | [yaron2](https://github.com/yaron2) | Microsoft |
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
| Aarthi Saravanakumar | [Aarthisk](https://github.com/Aarthisk) | Microsoft |
| Yaron Schneider | [yaron2](https://github.com/yaron2) | Microsoft |
| Satish Ranjan | [SatishRanjan](https://github.com/SatishRanjan) | Microsoft |

Copy link
Member

Choose a reason for hiding this comment

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

Yeah per earlier conversations for this specific markdown we are going to try to keep to just the active maintainers. I think it does make sense to give credit to initial ones, potentially even another section in this? FWIW in the CNCF proposal I don't plan on changing that. This would be more a reference for folks needing insight in current day to day of KEDA. cncf/toc#383

Copy link
Member Author

Choose a reason for hiding this comment

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

That makes sense to me, does that sound fair to you @yaron2 @Aarthisk ?

Copy link
Contributor

Choose a reason for hiding this comment

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

yep

Copy link
Member Author

Choose a reason for hiding this comment

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

Introduced alumni and added @yaron2 & @Aarthisk to that but will wait for @Aarthisk to agree on that.

| Ahmed ElSayed | [ahmelsayed](https://github.com/ahmelsayed) | Microsoft |
| Zbynek Roubalik | [zroubalik](https://github.com/zroubalik) | Red Hat |
| Ben Browning | [bbrowning](https://github.com/bbrowning) | Red Hat |
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
| Ben Browning | [bbrowning](https://github.com/bbrowning) | Red Hat |

Copy link
Member Author

Choose a reason for hiding this comment

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

Is that ok for you @zroubalik?

Copy link
Member Author

Choose a reason for hiding this comment

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

Introduced alumni and added @bbrowning to that but will wait for @bbrowning or @zroubalik to agree on that.

| Tom Kerkhove | [tomkerkhove](https://github.com/tomkerkhove) | Codit |