Skip to content

Add documentation for VS Profiler - #2672

Merged
adamsitnik merged 2 commits into
dotnet:masterfrom
karpinsn:master
Dec 9, 2024
Merged

Add documentation for VS Profiler#2672
adamsitnik merged 2 commits into
dotnet:masterfrom
karpinsn:master

Conversation

@karpinsn

@karpinsn karpinsn commented Dec 6, 2024

Copy link
Copy Markdown
Contributor
  • This change adds documentation for the Visual Studio profiler. There is a new document under features that explains how to use the nuget package and a new sample under IntroVisualStudioDiagnoser. A new package reference has also been added to the samples project to pull in the Visual Studio specific diagnosers for the sample project.

- This change adds documentation for the Visual Studio profiler.
  There is a new document under features that explains how to use the
  nuget package and a new sample under IntroVisualStudioDiagnoser. A new
  package reference has also been added to the samples project to pull
  in the Visual Studio specific diagnosers for the sample project.

@adamsitnik adamsitnik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It work very nice, the doc is also great. And there is no need to run as an Admin which is neat!

Thank you for your contribution @karpinsn !

Comment thread samples/BenchmarkDotNet.Samples/IntroVisualStudioDiagnoser.cs
Comment thread docs/articles/features/vsprofiler.md

@adamsitnik adamsitnik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Again thank you for your contribution @karpinsn !

@adamsitnik
adamsitnik merged commit 6367ad8 into dotnet:master Dec 9, 2024
@adamsitnik adamsitnik added this to the v0.14.1 milestone Dec 9, 2024
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