Skip to content

Commit 9c67714

Browse files
authored
Update CODEOWNERS to include doc writers and reduce duplication (#60017)
* Update CODEOWNERS to include doc writers and reduce duplication This PR adds @Rick-Anderson and @tdykstra as CODEOWNERS for `/**/PublicAPI.*Shipped.txt`, so they get notified for any PR that adds new API so they can review the doc comments. Alternatively, they could join @aspnet-api-review, but that may be used for more than just PRs. I also took the opportunity to remove some redundancy with the assumption that the [last matched pattern wins](https://graphite.dev/guides/in-depth-guide-github-codeowners#patterns-and-precedence-in-the-codeowners-file). If we find that's not the case, we can revert that part later and add @Rick-Anderson and @tdykstra to all the relevant lines. * Remove dotnet-maestro-bot from CODEOWNERS
1 parent 16eac9f commit 9c67714

File tree

1 file changed

+5
-18
lines changed

1 file changed

+5
-18
lines changed

.github/CODEOWNERS

+5-18
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
# Users referenced in this file will automatically be requested as reviewers for PRs that modify the given paths.
22
# See https://help.github.com/articles/about-code-owners/
33

4-
/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review
54
/global.json @dotnet/aspnet-build @wtgodbe
65
/.azure/ @dotnet/aspnet-build @wtgodbe
76
/.azuredevops/ @dotnet/aspnet-build @wtgodbe
@@ -13,39 +12,27 @@
1312
/.github/workflows/ @dotnet/aspnet-build @wtgodbe
1413
/docs/ @captainsafia @mkArtakMSFT
1514
/eng/ @dotnet/aspnet-build @wtgodbe
16-
/eng/common/ @dotnet-maestro-bot
17-
/eng/Versions.props @dotnet-maestro-bot @dotnet/aspnet-build @wtgodbe
18-
/eng/Version.Details.xml @dotnet-maestro-bot @dotnet/aspnet-build @wtgodbe
15+
/eng/Versions.props @dotnet/aspnet-build @wtgodbe
16+
/eng/Version.Details.xml @dotnet/aspnet-build @wtgodbe
1917
/eng/SourceBuild* @dotnet/source-build
2018
/src/Caching/ @captainsafia @halter73 @mgravell
21-
/src/Caching/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @captainsafia @halter73 @mgravell
2219
/src/Components/ @dotnet/aspnet-blazor-eng
23-
/src/Components/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @dotnet/aspnet-blazor-eng
2420
/src/DefaultBuilder/ @halter73
25-
/src/DefaultBuilder/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review
2621
/src/Grpc/ @JamesNK @captainsafia @mgravell
27-
/src/Grpc/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @JamesNK @captainsafia @mgravell
2822
/src/Hosting/ @halter73
29-
/src/Hosting/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review
3023
/src/Http/ @BrennanConroy @halter73 @captainsafia
31-
/src/Http/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @BrennanConroy
3224
/src/Http/Routing/ @halter73
33-
/src/Http/Routing/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @halter73
3425
/src/HttpClientFactory/ @captainsafia @halter73
35-
/src/HttpClientFactory/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @captainsafia @halter73
3626
/src/Installers/ @dotnet/aspnet-build @wtgodbe
3727
/src/JSInterop/ @dotnet/aspnet-blazor-eng
3828
/src/Middleware/ @BrennanConroy
39-
/src/Middleware/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @BrennanConroy
4029
/src/Mvc/ @dotnet/minimal-apis
41-
/src/Mvc/Mvc.ApiExplorer @captainsafia @halter73 @brunolins16
42-
/src/Mvc/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @dotnet/aspnet-blazor-eng
30+
/src/Mvc/Mvc.ApiExplorer @captainsafia @halter73
4331
/src/OpenApi @captainsafia @dotnet/minimal-apis
4432
/src/ProjectTemplates/Web.ProjectTemplates/content/BlazorServerWeb-CSharp/ @dotnet/aspnet-blazor-eng
4533
/src/ProjectTemplates/Web.ProjectTemplates/content/ComponentsWebAssembly-CSharp/ @dotnet/aspnet-blazor-eng
46-
/src/Security/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review
34+
/src/Security/ @halter73
4735
/src/Servers/ @halter73 @BrennanConroy @JamesNK @mgravell
48-
/src/Servers/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @halter73 @BrennanConroy @JamesNK @mgravell
4936
/src/SignalR/ @BrennanConroy @halter73
50-
/src/SignalR/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @BrennanConroy @halter73
5137
/src/submodules @dotnet/aspnet-build @wtgodbe
38+
/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @Rick-Anderson @tdykstra

0 commit comments

Comments
 (0)