Jelajahi Sumber

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
Stephen Halter 1 tahun lalu
induk
melakukan
9c677146b0
1 mengubah file dengan 5 tambahan dan 18 penghapusan
  1. 5 18
      .github/CODEOWNERS

+ 5 - 18
.github/CODEOWNERS

@@ -1,7 +1,6 @@
 # Users referenced in this file will automatically be requested as reviewers for PRs that modify the given paths.
 # Users referenced in this file will automatically be requested as reviewers for PRs that modify the given paths.
 # See https://help.github.com/articles/about-code-owners/
 # See https://help.github.com/articles/about-code-owners/
 
 
-/**/PublicAPI.*Shipped.txt                                                        @dotnet/aspnet-api-review
 /global.json                                                                      @dotnet/aspnet-build @wtgodbe
 /global.json                                                                      @dotnet/aspnet-build @wtgodbe
 /.azure/                                                                          @dotnet/aspnet-build @wtgodbe
 /.azure/                                                                          @dotnet/aspnet-build @wtgodbe
 /.azuredevops/                                                                    @dotnet/aspnet-build @wtgodbe
 /.azuredevops/                                                                    @dotnet/aspnet-build @wtgodbe
@@ -13,39 +12,27 @@
 /.github/workflows/                                                               @dotnet/aspnet-build @wtgodbe
 /.github/workflows/                                                               @dotnet/aspnet-build @wtgodbe
 /docs/                                                                            @captainsafia @mkArtakMSFT
 /docs/                                                                            @captainsafia @mkArtakMSFT
 /eng/                                                                             @dotnet/aspnet-build @wtgodbe
 /eng/                                                                             @dotnet/aspnet-build @wtgodbe
-/eng/common/                                                                      @dotnet-maestro-bot
-/eng/Versions.props                                                               @dotnet-maestro-bot @dotnet/aspnet-build @wtgodbe
-/eng/Version.Details.xml                                                          @dotnet-maestro-bot @dotnet/aspnet-build @wtgodbe
+/eng/Versions.props                                                               @dotnet/aspnet-build @wtgodbe
+/eng/Version.Details.xml                                                          @dotnet/aspnet-build @wtgodbe
 /eng/SourceBuild*                                                                 @dotnet/source-build
 /eng/SourceBuild*                                                                 @dotnet/source-build
 /src/Caching/                                                                     @captainsafia @halter73 @mgravell
 /src/Caching/                                                                     @captainsafia @halter73 @mgravell
-/src/Caching/**/PublicAPI.*Shipped.txt                                            @dotnet/aspnet-api-review @captainsafia @halter73 @mgravell
 /src/Components/                                                                  @dotnet/aspnet-blazor-eng
 /src/Components/                                                                  @dotnet/aspnet-blazor-eng
-/src/Components/**/PublicAPI.*Shipped.txt                                         @dotnet/aspnet-api-review @dotnet/aspnet-blazor-eng
 /src/DefaultBuilder/                                                              @halter73
 /src/DefaultBuilder/                                                              @halter73
-/src/DefaultBuilder/**/PublicAPI.*Shipped.txt                                     @dotnet/aspnet-api-review
 /src/Grpc/                                                                        @JamesNK @captainsafia @mgravell
 /src/Grpc/                                                                        @JamesNK @captainsafia @mgravell
-/src/Grpc/**/PublicAPI.*Shipped.txt                                               @dotnet/aspnet-api-review @JamesNK @captainsafia @mgravell
 /src/Hosting/                                                                     @halter73
 /src/Hosting/                                                                     @halter73
-/src/Hosting/**/PublicAPI.*Shipped.txt                                            @dotnet/aspnet-api-review
 /src/Http/                                                                        @BrennanConroy @halter73 @captainsafia
 /src/Http/                                                                        @BrennanConroy @halter73 @captainsafia
-/src/Http/**/PublicAPI.*Shipped.txt                                               @dotnet/aspnet-api-review @BrennanConroy
 /src/Http/Routing/                                                                @halter73
 /src/Http/Routing/                                                                @halter73
-/src/Http/Routing/**/PublicAPI.*Shipped.txt                                       @dotnet/aspnet-api-review @halter73
 /src/HttpClientFactory/                                                           @captainsafia @halter73
 /src/HttpClientFactory/                                                           @captainsafia @halter73
-/src/HttpClientFactory/**/PublicAPI.*Shipped.txt                                  @dotnet/aspnet-api-review @captainsafia @halter73
 /src/Installers/                                                                  @dotnet/aspnet-build @wtgodbe
 /src/Installers/                                                                  @dotnet/aspnet-build @wtgodbe
 /src/JSInterop/                                                                   @dotnet/aspnet-blazor-eng
 /src/JSInterop/                                                                   @dotnet/aspnet-blazor-eng
 /src/Middleware/                                                                  @BrennanConroy
 /src/Middleware/                                                                  @BrennanConroy
-/src/Middleware/**/PublicAPI.*Shipped.txt                                         @dotnet/aspnet-api-review @BrennanConroy
 /src/Mvc/                                                                         @dotnet/minimal-apis
 /src/Mvc/                                                                         @dotnet/minimal-apis
-/src/Mvc/Mvc.ApiExplorer                                                          @captainsafia @halter73 @brunolins16
-/src/Mvc/**/PublicAPI.*Shipped.txt                                                @dotnet/aspnet-api-review @dotnet/aspnet-blazor-eng
+/src/Mvc/Mvc.ApiExplorer                                                          @captainsafia @halter73
 /src/OpenApi                                                                      @captainsafia @dotnet/minimal-apis
 /src/OpenApi                                                                      @captainsafia @dotnet/minimal-apis
 /src/ProjectTemplates/Web.ProjectTemplates/content/BlazorServerWeb-CSharp/        @dotnet/aspnet-blazor-eng
 /src/ProjectTemplates/Web.ProjectTemplates/content/BlazorServerWeb-CSharp/        @dotnet/aspnet-blazor-eng
 /src/ProjectTemplates/Web.ProjectTemplates/content/ComponentsWebAssembly-CSharp/  @dotnet/aspnet-blazor-eng
 /src/ProjectTemplates/Web.ProjectTemplates/content/ComponentsWebAssembly-CSharp/  @dotnet/aspnet-blazor-eng
-/src/Security/**/PublicAPI.*Shipped.txt                                           @dotnet/aspnet-api-review
+/src/Security/                                                                    @halter73
 /src/Servers/                                                                     @halter73 @BrennanConroy @JamesNK @mgravell
 /src/Servers/                                                                     @halter73 @BrennanConroy @JamesNK @mgravell
-/src/Servers/**/PublicAPI.*Shipped.txt                                            @dotnet/aspnet-api-review @halter73 @BrennanConroy @JamesNK @mgravell
 /src/SignalR/                                                                     @BrennanConroy @halter73
 /src/SignalR/                                                                     @BrennanConroy @halter73
-/src/SignalR/**/PublicAPI.*Shipped.txt                                            @dotnet/aspnet-api-review @BrennanConroy @halter73
 /src/submodules                                                                   @dotnet/aspnet-build @wtgodbe
 /src/submodules                                                                   @dotnet/aspnet-build @wtgodbe
+/**/PublicAPI.*Shipped.txt                                                        @dotnet/aspnet-api-review @Rick-Anderson @tdykstra