Skip to content

Conversation

Copilot AI review requested due to automatic review settings December 23, 2025 23:18
@gewarren gewarren requested review from a team and tommcdon as code owners December 23, 2025 23:18
@dotnetrepoman dotnetrepoman bot added this to the December 2025 milestone Dec 23, 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 applies an Acrolinx pass to improve the writing style and consistency across three diagnostics tutorial documents. The changes focus on grammar corrections, present tense usage, contraction standardization, and improved clarity in technical documentation.

Key changes:

  • Corrects article indefinite article usage ("An" to "A" in descriptions)
  • Standardizes contractions (doesn't, you'll, it's) following Microsoft style guidelines
  • Converts future tense to present tense for instructions and descriptions
  • Improves list formatting with colons and proper punctuation
  • Enhances clarity by breaking up long sentences and improving flow

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
docs/core/diagnostics/observability-prgrja-example.md Style improvements including grammar fixes (doesn't vs does not), present tense adoption, URL capitalization, list formatting with colons, and sentence restructuring for clarity
docs/core/diagnostics/observability-otlp-example.md Grammar corrections, contraction standardization, present tense conversion, Docker capitalization, and improved sentence flow throughout the tutorial
docs/core/diagnostics/observability-applicationinsights.md Corrected description article, standardized verb form (Set up vs Setup), and improved Azure portal capitalization

@gewarren gewarren merged commit ba14c80 into dotnet:main Dec 24, 2025
9 checks passed
@gewarren gewarren deleted the acro-diagnostics branch December 24, 2025 01:32
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