Skip to content

Conversation

@BillWagner
Copy link
Member

@BillWagner BillWagner commented Dec 12, 2025

@BillWagner BillWagner marked this pull request as ready for review December 12, 2025 16:51
@BillWagner BillWagner requested a review from a team as a code owner December 12, 2025 16:51
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 consolidates property-related compiler errors and warnings into a single comprehensive diagnostic page, improving documentation discoverability and maintainability by bringing together scattered error documentation from individual files and the misc folder.

Key changes:

  • Creates a new consolidated page property-declaration-errors.md covering 41 error codes related to property declarations, including readonly properties, auto-implemented properties, field-backed properties, required members, ref-returning properties, and property initializers
  • Removes field-backed property documentation from partial-declarations.md (CS9258, CS9263, CS9264, CS9266, CS9273) since these errors are more closely related to property declarations than partial declarations
  • Updates TOC and redirection mappings to ensure all deleted error pages redirect to the new consolidated page

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/language-reference/compiler-messages/property-declaration-errors.md New consolidated documentation covering 41 property-related compiler errors and warnings with detailed explanations and resolution steps
docs/csharp/language-reference/compiler-messages/partial-declarations.md Removes field-backed property section (moved to property-declaration-errors.md) and updates error code lists
docs/csharp/language-reference/toc.yml Adds new property declarations entry with displayName listing all covered error codes and moves field-backed property codes from partial declarations section
docs/csharp/misc/sorry-we-don-t-have-specifics-on-this-csharp-error.md Removes 22 error codes that now have proper documentation in the consolidated page
.openpublishing.redirection.csharp.json Adds redirects for 8 deleted individual error pages (cs0200.md, cs0545.md, cs0571.md, cs0840.md, cs1014.md, cs1043.md, cs8145.md, cs9036.md)
docs/csharp/misc/cs0200.md, cs1014.md, cs1043.md Deleted individual error pages now consolidated into property-declaration-errors.md
docs/csharp/language-reference/compiler-messages/cs0545.md, cs0571.md, cs0840.md, cs8145.md, cs9036.md Deleted individual error pages now consolidated into property-declaration-errors.md
docs/csharp/language-reference/compiler-messages/snippets/partial-declarations/ Removed unused snippet files (CS0759Examples.cs and project file) as the examples were replaced with consolidated guidance
.github/prompts/error-consolidation.md Updated prompt configuration to target property declaration errors

BillWagner and others added 2 commits December 12, 2025 12:50
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@BillWagner BillWagner merged commit 35cbb6e into dotnet:main Dec 12, 2025
9 checks passed
@BillWagner BillWagner deleted the required-property-diagnostics branch December 12, 2025 19:44
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.

Add C# 11 errors for required members and related topics

2 participants