-
Notifications
You must be signed in to change notification settings - Fork 226
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
nonspec: Add trishankatdatadog to MAINTAINERS.md #1176
Conversation
Signed-off-by: Trishank Karthik Kuppusamy <[email protected]>
✅ Deploy Preview for slsa canceled.
|
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.
Happy to have 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.
Happy to have you
Thank you very much! 🙂 Let me request to join this team. Once I have joined, I think it's safe to merge this PR. |
I will add you once I get back to a computer with my yubikey |
Done! |
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.
Seems reasonable to me. Thanks for your contributions!
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.
It will be good to have you on board!
Signed-off-by: Arnaud J Le Hors <[email protected]>
We have now reached the required majority of maintainers approving this proposal. So, I'm merging it. |
Following discussion with @TomHennen and @mlieberman85, I'd like to propose making myself a maintainer in order to expedite PR reviews.
I believe I meet the requirements listed here. Having said that, feel free to disagree!