Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 5, 2026

Addresses style feedback on the Overview section's second paragraph, which used passive voice and complex phrasing that didn't follow documentation guidelines.

Changes:

  • Converted passive constructions ("provides for generality") to active voice with "you" form
  • Simplified verbose phrases ("at the expense of some tedium" → "add verbose syntax")
  • Applied conversational tone with present tense throughout

Before:

The ability of a class or struct in C# to have more than one constructor provides for 
generality, but at the expense of some tedium in the declaration syntax, because the 
constructor input and the class state need to be cleanly separated.

After:

When you create a class or struct with multiple constructors, you gain flexibility, 
but you also add verbose syntax. You must cleanly separate the constructor input from 
the class state.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.


Internal previews

📄 File 🔗 Preview link
docs/fundamentals/code-analysis/style-rules/ide0290.md docs/fundamentals/code-analysis/style-rules/ide0290

Converted passive voice paragraph to active voice with conversational tone per style guidelines. Uses "you" form and simpler, more direct language.

Co-authored-by: gewarren <24882762+gewarren@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix documentation on primary constructor usage rationale Rewrite primary constructor rationale in active voice Jan 5, 2026
Copilot AI requested a review from gewarren January 5, 2026 16:33
@gewarren gewarren marked this pull request as ready for review January 5, 2026 16:36
@gewarren gewarren requested a review from a team as a code owner January 5, 2026 16:36
@gewarren gewarren merged commit ba6f51a into gewarren-patch-5 Jan 5, 2026
9 of 10 checks passed
@gewarren gewarren deleted the copilot/sub-pr-50741 branch January 5, 2026 16:36
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