Skip to content

Move includes only used once into their articles#51944

Merged
adegeo merged 4 commits intomainfrom
adegeo/dateonly-and-timeonly-includes/50697
Feb 27, 2026
Merged

Move includes only used once into their articles#51944
adegeo merged 4 commits intomainfrom
adegeo/dateonly-and-timeonly-includes/50697

Conversation

@adegeo
Copy link
Contributor

@adegeo adegeo commented Feb 26, 2026

Copilot AI review requested due to automatic review settings February 26, 2026 23:50
@adegeo adegeo requested a review from a team as a code owner February 26, 2026 23:50
@dotnetrepoman dotnetrepoman bot added this to the February 2026 milestone Feb 26, 2026
@adegeo adegeo enabled auto-merge (squash) February 26, 2026 23:50
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 eliminates unnecessary include files by inlining content that was only referenced once. The linked issue (#50697) identified two include files (dateonly-and-timeonly-serialization.md and dateonly-and-timeonly-serialization-cs.md) that were each used in only one location, making them candidates for removal.

Changes:

  • Deleted two single-use include files in the docs/standard/datetime/includes/ directory
  • Inlined the DateOnly and TimeOnly serialization content into the two articles that referenced them
  • Added HTML comments noting that content is duplicated between the two articles

Reviewed changes

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

File Description
docs/standard/datetime/system-text-json-support.md Replaced include reference with inline content (C# only)
docs/standard/datetime/how-to-use-dateonly-timeonly.md Replaced include reference with inline content (C# and VB)
docs/standard/datetime/includes/dateonly-and-timeonly-serialization.md Deleted (was only used once)
docs/standard/datetime/includes/dateonly-and-timeonly-serialization-cs.md Deleted (was only used once)

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 @adegeo

Let's :shipit:

@adegeo adegeo disabled auto-merge February 27, 2026 18:10
@adegeo adegeo enabled auto-merge (squash) February 27, 2026 18:10
@adegeo adegeo merged commit b5dd985 into main Feb 27, 2026
11 checks passed
@adegeo adegeo deleted the adegeo/dateonly-and-timeonly-includes/50697 branch February 27, 2026 18:17
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.

Two odd includes - Investigate

3 participants