Remap author metadata - #37745
Remap author metadata#37745guardrex wants to merge 17 commits into
Conversation
|
Adding @gewarren as well. @guardrex This is a worthy project, and the right time to work on it. I haven't given this is thorough detailed review yet, as our org is working through what's coming next this week. In the meantime, my main suggestion is to use this task to migrate to using docfx.json's folder mapping to simplify the metadata. See https://github.com/dotnet/docs/blob/main/docfx.json#L400 for an example of how we do this in the .NET docs repo. This structure makes it much easier to see who's responsible for what, and make changes as roles change. Also, any metadata in the article overrides the folder setting at build time, so we can support individual articles by other authors. Can you start by updating this PR to make reasonable folder defaults in docfx.json, and remove the article specific tags that are redundant with the default? That will make it much easier to review. |
|
That makes sense! When Wade, Tom, and I had the articles split, it was mostly by folder along the lines of major subject areas. We could've adopted the I'll update this now, and then we can take another look. I'll ping you back. |
…ub.com/dotnet/AspNetCore.Docs into guardrex/remap-article-author-metadata
|
@BillWagner @gewarren ... Break out the 👁️ drops! 😄 This one is a beast to review. Even if this is all good thus far, we still need @danroth27 to look at my OP for the odd assignments. I went ahead and moved all of the assignments to the |
|
I fixed the glob patterns per the guidelines at https://dotnet.github.io/docfx/docs/config.html. |
|
@danroth27 @BillWagner @gewarren ... Pinging to see if we can get this one done and merged quickly. 🏃♂️ It's probably going to 💥 with a merge conflict every time another PR is merged because it touches all of the articles. |
3 participants
Fixes #37744
Dan, Bill ... I'd like to receive the community issue pings if I'm handling the day-to-day operations for now or until the end of the calendar year (and Bill, I made you the
ms.author). Close this PR if there are other plans in place.WRT other authors: If we ignore the following authors ...
... let me know if the following are OK to leave in place or what you want them changed to ...
alexwolfmsft: aspnetcore\host-and-deploy\scaling-aspnet-apps\scaling-aspnet-apps.mdshirhatti: aspnetcore\fundamentals\servers\kestrel\diagnostics.mdInternal previews
Toggle expand/collapse
Note
This table shows the first 50 preview links (sorted alphabetically by file path) found in the OPS build report. For the full list, select OpenPublishing.Build Details within checks.
Build report