Update default reviewers#15355
Conversation
|
Thank you, @mairaw! |
|
What are you doing here? 😄 Go relax! |
|
I just happened to be checking email. |
BillWagner
left a comment
BillWagner
left a comment
There was a problem hiding this comment.
This LGTM.
I had one question if an area is related. I'll defer to @gewarren if she wants it changed.
| # Assembly | ||
| /docs/standard/assembly/** @gewarren | ||
| # Asynchronous Programming Patterns | ||
| /docs/standard/asynchronous-programming-patterns/** @gewarren |
There was a problem hiding this comment.
I've got responsibility for async in C# (along with threading). This is related. Also, happy to work with @gewarren to make sure these areas are seamless.
There was a problem hiding this comment.
I think this makes sense @BillWagner. We should ensure that no one is overwhelmed by the number of PRs.
gewarren
left a comment
gewarren
left a comment
There was a problem hiding this comment.
There's a handful of folders missing from under docs/framework, but is that just because they'll fall under the umbrella of the parent folder code owner?
Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
6 participants
Part I of our retirement gift to @rpetrusha 🎁
This only sets the default reviewers on the codeowners file.