Skip to content

Conversation

@gewarren
Copy link
Contributor

@gewarren gewarren commented Dec 8, 2025

This rule causes too much noise and false positives in Copilot pull request reviews.


Internal previews

📄 File 🔗 Preview link
.github/agents/docseditor.agent.md Article Editing Instructions for LLMs
.github/instructions/Markdown.WritingStyle.instructions.md Markdown Writing Style Instructions

Removed non-negotiable rule about list item punctuation.
@gewarren gewarren requested a review from a team as a code owner December 8, 2025 18:03
Copilot AI review requested due to automatic review settings December 8, 2025 18:03
@dotnetrepoman dotnetrepoman bot added this to the December 2025 milestone Dec 8, 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 removes the guidance requiring periods at the end of list items that exceed three words. The change addresses issues with noise and false positives in Copilot pull request reviews.

  • Removes the mandatory rule about ending list items with periods from writing style instructions
  • Updates the docseditor agent configuration to remove period-related list guidance
  • Cleans up redundant punctuation rules across instruction files

Reviewed changes

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

File Description
.github/instructions/Markdown.WritingStyle.instructions.md Removes the mandatory rule requiring periods for list items over three words
.github/agents/docseditor.agent.md Removes three references to the period-in-lists rule from critical rules, punctuation fixes, and list rules sections

@gewarren gewarren merged commit f728d04 into main Dec 8, 2025
19 checks passed
@gewarren gewarren deleted the gewarren-patch-4 branch December 8, 2025 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants