-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Update CODEOWNERS #3852
Update CODEOWNERS #3852
Conversation
@@ -24,4 +24,4 @@ | |||
|
|||
# The following lines are used by GitHub to automatically recommend reviewers. | |||
|
|||
* @abertelrud @neonichu @tomerd @friedbunny | |||
* @abertelrud @neonichu @tomerd @elsh |
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.
Intentional replacement (summary says to add)?
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.
Also, should the section above be updated while it's being edited (with the N
, E
, D
, etc)?
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 thought it was just comments but I think Rick said it gets parsed by GitHub.
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.
added. I think it makes sense to keep Daniel and Ankit info in that section since they show up in commit history, correct?
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.
Intentional replacement (summary says to add)?
yes I believe @friedbunny would be less involved in these going forward
@@ -24,4 +24,4 @@ | |||
|
|||
# The following lines are used by GitHub to automatically recommend reviewers. | |||
|
|||
* @abertelrud @neonichu @tomerd @friedbunny | |||
* @abertelrud @neonichu @tomerd @elsh |
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.
Also, should the section above be updated while it's being edited (with the N
, E
, D
, etc)?
@@ -24,4 +24,4 @@ | |||
|
|||
# The following lines are used by GitHub to automatically recommend reviewers. | |||
|
|||
* @abertelrud @neonichu @tomerd @friedbunny | |||
* @abertelrud @neonichu @tomerd @elsh |
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 thought it was just comments but I think Rick said it gets parsed by GitHub.
add @elsh to CODEOWNERS