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
[Security Group] add github names of security group members. #69304
Conversation
Also drop phabricator names as we no longer use phabricator.
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'm okay with at least my changed line, and @emaste's, I cannot vouch for any of the others though :)
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.
My name is correct. LGTM on my side.
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.
Mine is correct
LGTM for my personal info |
Co-authored-by: Andy Kaylor <andrew.kaylor@intel.com>
LGTM for the @gburgessiv line :) Thanks for this! |
Mine is good |
LGTM for my github handle |
Mine and @cuviper's are correct. |
Adding @tpenge |
Also drop phabricator names as we no longer use phabricator.