Skip to content

Move ms.author to doc team - Update ms.reviewer - Verify values active#37073

Merged
tdykstra merged 5 commits intomainfrom
copilot/set-ms-author-to-wpickett
May 4, 2026
Merged

Move ms.author to doc team - Update ms.reviewer - Verify values active#37073
tdykstra merged 5 commits intomainfrom
copilot/set-ms-author-to-wpickett

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 29, 2026 •
edited by github-actions Bot
Loading

Fixes #37072

~60 articles had ms.author set to inactive or non-doc-team members (stevesa, tasou, scaddie, dmkorolev, jeliknes, brolivei, etc.), meaning doc team wasn't receiving notifications for these articles.

Set all non-compliant ms.author values to wpickett as the default. Affected areas:

  • aspnetcore/breaking-changes/5/ — 30 files (previously scaddie)
  • aspnetcore/breaking-changes/6/ — 7 files (previously scaddie)
  • aspnetcore/migration/fx-to-core/ — 11 files (previously tasou)
  • aspnetcore/fundamentals/ — 4 files (various)
  • aspnetcore/security/ — 2 files
  • Remaining scattered files — 6 files (stevesa, dmkorolev, marcgravell, chrross, alexwolf, chenyl, bdorrans)

Valid values retained: wpickett, tdykstra, guardrex.

For active development team members still on these respective areas whose ms.author was replaced, ms.reviewer was added in alphabetical order in the metadata so they continue to receive review notifications:

  • wigodbe → 1 file (w3c-logger/index.md)
  • midenn → 1 file (native-aot-tutorial.md)
  • dmkorolev → 1 file (asp0028.md)
  • stevesa → 1 file (angular.md)
  • brolivei → 1 file (responses.md)
  • jeliknes → 2 files (apis.md, how-to-choose-identity-solution.md)
  • chrross → 1 file (http3.md)
  • tasou → 11 files (all migration/fx-to-core/ articles)

Internal previews

Note

This table shows preview links for the 30 files with the most changes. For preview links for other files in this PR, select OpenPublishing.Build Details within checks.

Internal previews

Toggle expand/collapse
📄 File 🔗 Preview link
aspnetcore/breaking-changes/5/authentication-aad-packages-obsolete.md Authentication: AzureAD.UI and AzureADB2C.UI APIs and packages marked obsolete
aspnetcore/breaking-changes/5/authorization-resource-in-endpoint-routing.md "Breaking change: Authorization: Resource in endpoint routing is HttpContext"
aspnetcore/breaking-changes/5/azure-integration-packages-removed.md Azure: Microsoft-prefixed Azure integration packages removed
aspnetcore/breaking-changes/5/blazor-browser-support-updated.md aspnetcore/breaking-changes/5/blazor-browser-support-updated
aspnetcore/breaking-changes/5/blazor-components-trim-insignificant-whitespace.md Blazor: Insignificant whitespace trimmed from components at compile time
aspnetcore/breaking-changes/5/blazor-jsobjectreference-to-internal.md "Breaking change: Blazor: JSObjectReference and JSInProcessObjectReference types changed to internal"
aspnetcore/breaking-changes/5/blazor-packages-target-framework-changed.md Blazor: Target framework of NuGet packages changed
aspnetcore/breaking-changes/5/blazor-protectedbrowserstorage-moved.md aspnetcore/breaking-changes/5/blazor-protectedbrowserstorage-moved
aspnetcore/breaking-changes/5/blazor-rendertreeframe-fields-become-properties.md Blazor: RenderTreeFrame readonly public fields have become properties
aspnetcore/breaking-changes/5/blazor-routing-logic-changed.md "Breaking change: Blazor: Changes to routing logic in Blazor apps"
aspnetcore/breaking-changes/5/blazor-static-web-assets-validation-logic-updated.md Blazor: Updated validation logic for static web assets
aspnetcore/client-side/spa/angular.md aspnetcore/client-side/spa/angular
aspnetcore/diagnostics/asp0028.md ASP0028: Consider using IPAddress.IPv6Any instead of IPAddress.Any
aspnetcore/fundamentals/aot/native-aot-tutorial.md Tutorial: Publish an ASP.NET Core app using Native AOT
aspnetcore/fundamentals/apis.md APIs overview
aspnetcore/fundamentals/minimal-apis/responses.md How to create responses in Minimal API apps
aspnetcore/fundamentals/w3c-logger/index.md aspnetcore/fundamentals/w3c-logger/index
aspnetcore/host-and-deploy/iis/http3.md Use ASP.NET Core with HTTP/3 on IIS
aspnetcore/migration/fx-to-core/areas/authentication.md Migrate ASP.NET Framework Authentication to ASP.NET Core
aspnetcore/migration/fx-to-core/areas/hosting.md .NET Generic Host in ASP.NET Framework
aspnetcore/migration/fx-to-core/areas/http-context.md Migrate ASP.NET Framework HttpContext to ASP.NET Core
aspnetcore/migration/fx-to-core/areas/http-handlers.md aspnetcore/migration/fx-to-core/areas/http-handlers
aspnetcore/migration/fx-to-core/areas/http-modules.md Migrate HTTP modules to ASP.NET Core middleware
aspnetcore/migration/fx-to-core/areas/index.md Complex migration scenarios - Deep dive areas
aspnetcore/migration/fx-to-core/areas/machine-key.md Migrate ASP.NET machineKey by using System.Web adapters
aspnetcore/migration/fx-to-core/areas/misc.md aspnetcore/migration/fx-to-core/areas/misc
aspnetcore/migration/fx-to-core/areas/owin.md OWIN integration
aspnetcore/migration/fx-to-core/inc/ab-testing.md A/B Testing Migrated Endpoints
aspnetcore/migration/fx-to-core/inc/blazor.md Enable ASP.NET Core Blazor Server support with Yarp in incremental migration
aspnetcore/security/how-to-choose-identity-solution.md aspnetcore/security/how-to-choose-identity-solution

Note

This table shows preview links for the 30 files with the most changes. For preview links for other files in this PR, select OpenPublishing.Build Details within checks.

Copilot AI linked an issue Apr 29, 2026 that may be closed by this pull request
Verify all ms.author values are set to doc team #37072
Closed
@wadepickett
Update ms.author to wpickett for all non-doc-team values
a0202a2 
Fixes #34939

Updated 60 .md files where ms.author was set to non-doc-team
members (stevesa, tasou, scaddie, dmkorolev, jeliknes, brolivei,
wigodbe, midenn, chenyl, chrross, alexwolf, marcgravell, bdorrans)
to use the default value of wpickett.

Agent-Logs-Url: https://github.com/dotnet/AspNetCore.Docs/sessions/ed9675e2-d141-4faa-a5d0-fa930714ce5b

Co-authored-by: wadepickett <10985336+wadepickett@users.noreply.github.com>
Copilot AI changed the title [WIP] Update ms.author values to set to doc team Update ms.author to doc team members across 60 articles Apr 29, 2026
Copilot AI requested a review from wadepickett April 29, 2026 16:58
@wadepickett
Copy link
Copy Markdown
Contributor

wadepickett commented Apr 29, 2026

@copilot,
There was a typo in the instructions for this PR.
For this item:
Valid values retained: wpickett, tdykstra, guardrx.
Should have been:
Valid values retained: wpickett, tdykstra, guardrex.

Revalidate the resulting PR changes are still correct and if not correct them, based on that corrected value of guardrex.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 29, 2026

@copilot,
There was a typo in the instructions for this PR.
For this item:
...

Verified — the typo didn't affect the PR. No files in the repo had ms.author: guardrx or ms.author: guardrex, so no guardrex files were incorrectly changed to wpickett. All 62 replacements were from genuinely non-doc-team values (scaddie, tasou, jeliknes, stevesa, etc.). The changes are correct as-is.

@wadepickett
Copy link
Copy Markdown
Contributor

wadepickett commented Apr 29, 2026

@copilot, for any of the .md articles in this PR that originally had ms.author: wigodbe. Add a ms.reviewer wigodbe so it appears in aphabetical order in the metadata. That way the this active development team member will continue to be notified for reviews going forward.

@wadepickett
Add ms.reviewer: wigodbe to w3c-logger article
45d7c77 
Agent-Logs-Url: https://github.com/dotnet/AspNetCore.Docs/sessions/31a26732-7566-418b-b223-789c2aeb14ae

Co-authored-by: wadepickett <10985336+wadepickett@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 29, 2026

@copilot, for any of the .md articles in this PR that originally had ms.author: wigodbe. Add a ms.reviewer wigodbe so it appears in aphabetical order in the metadata. That way the this active de...

Only one file had ms.author: wigodbe — aspnetcore/fundamentals/w3c-logger/index.md. Added ms.reviewer: wigodbe in alphabetical order in the metadata. See commit 45d7c77.

@wadepickett
Copy link
Copy Markdown
Contributor

wadepickett commented Apr 29, 2026

@copilot, for any of the .md articles in this PR that originally had ms.author: midenn. Add a ms.reviewer midenn so it appears in aphabetical order in the metadata. That way the this active development team member will continue to be notified for reviews going forward.

Also do the same for these values that were ms.author values originally in a .md article in this PR, each has already been verified as active and valid for ms.reviewer values for those cases:
dmkorolev
stevesa
brolivei
jeliknes
chrross
tasou

@wadepickett
Add ms.reviewer for active dev team members (midenn, dmkorolev, steve…
832ea44 
…sa, brolivei, jeliknes, chrross, tasou)

Agent-Logs-Url: https://github.com/dotnet/AspNetCore.Docs/sessions/b82512e8-2de2-4b7c-b25f-6f44361039fb

Co-authored-by: wadepickett <10985336+wadepickett@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 29, 2026

@copilot, for any of the .md articles in this PR that originally had ms.author: midenn. Add a ms.reviewer midenn so it appears in aphabetical order in the metadata. That way the this active develo...

Added ms.reviewer in alphabetical order for all requested values across 18 files:

  • midenn → 1 file (native-aot-tutorial.md)
  • dmkorolev → 1 file (asp0028.md)
  • stevesa → 1 file (angular.md)
  • brolivei → 1 file (responses.md)
  • jeliknes → 2 files (apis.md, how-to-choose-identity-solution.md)
  • chrross → 1 file (http3.md)
  • tasou → 11 files (all migration/fx-to-core/ articles)

See commit 832ea44.

@wadepickett wadepickett requested a review from tdykstra April 29, 2026 17:55
@wadepickett
Copy link
Copy Markdown
Contributor

wadepickett commented Apr 29, 2026 •
edited
Loading

@tdykstra, could you verify if any of these are from areas you are assigned or of course, would just like to have? If so, please do change the ms.author value to yourself. When you are done, review/approve. Thanks!

@wadepickett wadepickett marked this pull request as ready for review April 29, 2026 17:56
@wadepickett wadepickett changed the title Update ms.author to doc team members across 60 articles Update ms.author to doc team members - Update ms.reviewer Apr 29, 2026
@wadepickett wadepickett changed the title Update ms.author to doc team members - Update ms.reviewer Move ms.author to doc team - Update ms.reviewer - Verify values active Apr 29, 2026
@wadepickett
Copy link
Copy Markdown
Contributor

wadepickett commented May 1, 2026

@tdykstra, could use your input.

@tdykstra
Copy link
Copy Markdown
Contributor

tdykstra commented May 4, 2026 •
edited
Loading

@tdykstra, could use your input.

It all looks fine to me. Is there something in particular that you that you wanted me to comment on?

@tdykstra tdykstra merged commit 0885857 into main May 4, 2026
4 checks passed
@tdykstra tdykstra deleted the copilot/set-ms-author-to-wpickett branch May 4, 2026 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tdykstra tdykstra tdykstra approved these changes

@wadepickett wadepickett Awaiting requested review from wadepickett

Assignees

@wadepickett wadepickett

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Verify all ms.author values are set to doc team

3 participants

@wadepickett @tdykstra