Skip to content

Conversation

@kevingosse
Copy link
Contributor

@kevingosse kevingosse commented Nov 25, 2025

Summary

Copy/pasting commands from the mpgo documentation doesn't work because it uses the wrong type of dashes ( instead of -). It was probably originally written in Word or something.


Internal previews

📄 File 🔗 Preview link
docs/framework/tools/mpgo-exe-managed-profile-guided-optimization-tool.md Mpgo.exe (Managed Profile Guided Optimization Tool)

Copilot AI review requested due to automatic review settings November 25, 2025 09:26
@kevingosse kevingosse requested review from a team and gewarren as code owners November 25, 2025 09:26
@dotnetrepoman dotnetrepoman bot added this to the November 2025 milestone Nov 25, 2025
@dotnet-policy-service dotnet-policy-service bot added dotnet-framework/svc community-contribution Indicates PR is created by someone from the .NET community. labels Nov 25, 2025
Copilot finished reviewing on behalf of kevingosse November 25, 2025 09:27
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 fixes incorrect dash characters in the mpgo documentation that prevented users from successfully copy-pasting commands. The issue was caused by en-dashes (–) being used instead of hyphens (-) for command-line parameters throughout the document, likely from the original content being authored in Word or another word processor that auto-converts hyphens to en-dashes.

  • Corrected all command-line parameter dashes from en-dashes (–) to hyphens (-) in code examples and inline references
  • Fixed parameter references in prose text to use proper hyphens
  • Ensured all mpgo.exe command syntax examples are now copy-paste ready

Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @kevingosse, good catch, and sorry about this!

@gewarren gewarren enabled auto-merge (squash) November 25, 2025 14:31
@gewarren gewarren merged commit b3c7d87 into dotnet:main Nov 25, 2025
9 checks passed
@kevingosse kevingosse deleted the kevingosse-patch-1 branch November 25, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates PR is created by someone from the .NET community. dotnet-framework/svc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants