Skip to content

Conversation

@gewarren
Copy link
Contributor

Per ms.subservice meeting today.

@gewarren gewarren requested a review from a team as a code owner January 15, 2026 23:36
Copilot AI review requested due to automatic review settings January 15, 2026 23:36
@dotnetrepoman dotnetrepoman bot added this to the January 2026 milestone Jan 15, 2026
@gewarren gewarren requested a review from BillWagner January 15, 2026 23:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates file path patterns in docfx.json to include both .md and .yml file extensions using the {md,yml} brace expansion syntax. This ensures that metadata attributes apply to both Markdown documentation files and YAML files throughout the repository.

Changes:

  • Extended file patterns in social_image_url, feedback_product_url, ms.service, ms.subservice, author, ms.author, titleSuffix, no-loc, and ms.update-cycle sections to include .yml files
  • Reordered some entries in social_image_url and feedback_product_url for better organization
Comments suppressed due to low confidence (2)

docfx.json:324

  • This pattern uses *.* while other patterns in the same file have been updated to use {md,yml}. For consistency, this should be updated to docs/csharp/**/*.{md,yml}: "csharp"` to match the pattern used elsewhere in this PR.
                "docs/csharp/**/*.*": "csharp",

docfx.json:321

  • These patterns in the dev_langs section only match .md files. For consistency with the changes in this PR, they should be updated to include .yml files using the pattern {md,yml}.
                "docs/fsharp/**/**.md": [
                    "fsharp"
                ],
                "docs/visual-basic/**/**.md": [
                    "vb"
                ]

@gewarren gewarren requested a review from cmastr January 16, 2026 00:39
@gewarren gewarren enabled auto-merge (squash) January 16, 2026 03:07
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM @gewarren

Let's :shipit:

@gewarren gewarren merged commit 25c6e76 into dotnet:main Jan 16, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants