Skip to content

Conversation

@gewarren
Copy link
Contributor

@gewarren gewarren commented Nov 28, 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 systematically updates documentation formatting to prevent XML tags from being localized. The changes ensure XML/HTML tags are properly marked with backticks (`<tag>`) instead of backslash escapes (\<tag>), which prevents localization tools from translating the tag names themselves.

Key Changes:

  • Converted all escaped XML tags to backtick-formatted code tags (`<summary>` instead of \<summary>)
  • Removed trailing whitespace from YAML frontmatter fields (f1_keywords, helpviewer_keywords)
  • Cleaned up trailing spaces in code blocks and prose text
  • Updated keyboard input formatting to use semantic HTML (<kbd>Enter</kbd>)

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
docs/csharp/misc/cs1723.md Updated XML tag references and cleaned trailing whitespace
docs/csharp/misc/cs1712.md Reformatted XML tag references and removed trailing spaces
docs/csharp/misc/cs1592.md Updated <include> tag reference to backtick format
docs/csharp/misc/cs1590.md Updated <include> tag reference to backtick format
docs/csharp/misc/cs1589.md Updated <include> tag reference to backtick format
docs/csharp/misc/cs1574.md Converted <exception> tag reference from escaped to backtick format
docs/csharp/misc/cs1570.md Updated multiple XML tag references (<exception>, <seealso>, <include>) to backtick format
docs/csharp/language-reference/xmldoc/recommended-tags.md Systematically converted all section headers and tag references from escaped format to backticks
docs/csharp/language-reference/compiler-options/index.md Updated <compiler> element reference to backtick format
docs/csharp/language-reference/compiler-messages/cs1705.md Updated <dependentAssembly> element reference (partial - other tags still use backslashes)
docs/core/unmanaged-api/debugging/icordebug/icordebugprocess6-enablevirtualmodulesplitting-method.md Converted interface name references to backtick format
docs/core/tutorials/library-with-visual-studio-code.md Updated keyboard input to use semantic <kbd> tag
docs/core/tools/dependencies.md Updated <PackageReference> heading to backtick format
docs/core/extensions/best-practices-for-developing-world-ready-apps.md Updated <globalization> element reference to backtick format
docs/core/deploying/runtime-store.md Updated XML element references (<TargetManifestFiles>, <PublishWithAspNetCoreTargetManifest>) to backtick format
docs/core/compatibility/serialization/7.0/binaryformatter-apis-produce-errors.md Updated heading and references to <EnableUnsafeBinaryFormatterSerialization> to backtick format

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

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 also LGTM @gewarren

@gewarren gewarren enabled auto-merge (squash) December 2, 2025 00:33
@gewarren gewarren merged commit 0c8435c into dotnet:main Dec 2, 2025
9 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