Skip to content

Conversation

@gewarren gewarren requested a review from a team as a code owner October 27, 2025 23:21
Copilot AI review requested due to automatic review settings October 27, 2025 23:21
@dotnetrepoman dotnetrepoman bot added this to the October 2025 milestone Oct 27, 2025
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 PR addresses broken links and build suggestions by fixing cross-references and removing unnecessary metadata. The changes primarily focus on correcting API documentation references and updating internal links to use proper relative paths instead of absolute URLs.

Key changes:

  • Fixed incorrect System.Json.JsonObject cross-references to use the correct System.Text.Json.Nodes.JsonObject namespace
  • Updated internal Microsoft Learn links to use relative paths instead of absolute URLs
  • Removed obsolete author metadata field

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
docs/fundamentals/networking/telemetry/metrics.md Removed obsolete ms.author metadata field
docs/core/whats-new/dotnet-9/libraries.md Fixed incorrect cross-reference to JsonObject with correct namespace
docs/core/whats-new/dotnet-10/libraries.md Fixed incorrect cross-reference to JsonObject with correct namespace
docs/core/tools/telemetry.md Replaced absolute URL with relative link to breaking change documentation
docs/core/porting/versioning-sdk-msbuild-vs.md Converted absolute URLs to relative links for Visual Studio lifecycle pages
docs/core/extensions/httpclient-latency-extensions.md Changed absolute URL to relative link for options pattern documentation
docs/architecture/cloud-native/observability-patterns.md Removed external blog link reference

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 gewarren merged commit 4bfb1d3 into dotnet:main Oct 28, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants