Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 23, 2026

Review feedback identified missing periods in table cells and list items across MSTest analyzer rule category documentation, violating the Microsoft writing style guide requirement for proper punctuation of complete sentences.

Changes made:

  • Added periods to all table cell entries containing complete sentences (rule titles) across 4 category files
  • Added periods to all bullet list items with complete sentences
  • Files updated: design-rules.md, performance-rules.md, suppression-rules.md, usage-rules.md

Example:

Before:

| [MSTEST0004](mstest0004.md) | Public types should be test classes | Info | Yes |

- **[MSTEST0004](mstest0004.md)**: Keep helper classes internal, only test classes should be public

After:

| [MSTEST0004](mstest0004.md) | Public types should be test classes. | Info | Yes |

- **[MSTEST0004](mstest0004.md)**: Keep helper classes internal, only test classes should be public.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.


Internal previews

📄 File 🔗 Preview link
docs/core/testing/mstest-analyzers/design-rules.md docs/core/testing/mstest-analyzers/design-rules
docs/core/testing/mstest-analyzers/performance-rules.md docs/core/testing/mstest-analyzers/performance-rules
docs/core/testing/mstest-analyzers/suppression-rules.md docs/core/testing/mstest-analyzers/suppression-rules
docs/core/testing/mstest-analyzers/usage-rules.md docs/core/testing/mstest-analyzers/usage-rules

Co-authored-by: Evangelink <11340282+Evangelink@users.noreply.github.com>
Copilot AI changed the title [WIP] Update analyzers documentation based on review comments Add missing periods to tables and lists in MSTest analyzer documentation Jan 23, 2026
Copilot AI requested a review from Evangelink January 23, 2026 20:04
@Evangelink Evangelink marked this pull request as ready for review January 23, 2026 20:07
@meaghanlewis meaghanlewis merged commit b43c1a6 into dev/amauryleve/mstest-diagnostics Jan 23, 2026
11 checks passed
@meaghanlewis meaghanlewis deleted the copilot/sub-pr-51209 branch January 23, 2026 20:40
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.

3 participants