-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[LLVM][Maintainers] Add llvm-cov / Coverage maintainer #130590
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
Conversation
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.
Sorry for the delay. I can accept. Thanks for the nomination.
That said, I am still a reviewee with bunch of unapproved request. I'd really like more a few maintainers in this area.
More maintainers is always better :) Do you have any suggestions for other coverage maintainers? |
I'd like to nominate @evodius96 and @MaskRay , or to let them nominate others. |
I’d like to nominate evodius96 as well, if he’s willing to accept... I’d prefer not to be included myself, as I’m currently juggling other responsibilities. I hope I can help when I find some free time. |
I accept the nomination; though it helps to know there are other maintainers. Thanks! |
27b3c2b
to
5714e16
Compare
Sorry for the delay here, I've now added @evodius96 as well. |
|
||
Takumi Nakamura \ | ||
[email protected] (email), [chapuni](https://github.com/chapuni) (GitHub) \ | ||
Alan Philipps \ |
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.
Just a nit: last name is spelled "Phipps".
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.
Sorry about that, should be fixed in c3795ba.
I think @chapuni has been the most active in this area recently (mostly related to MC/DC). Would you be willing to be listed as the maintainer for this component?