Skip to content

Add activity user guide link in README. #1857

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

plmaheu
Copy link

@plmaheu plmaheu commented Feb 26, 2021

Fixes #.

Changes

Fix TBD in docs by adding link.

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Design discussion issue #
  • Changes in public API reviewed

@plmaheu plmaheu requested a review from a team February 26, 2021 20:17
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 26, 2021

CLA Signed

The committers are authorized under a signed CLA.

documented fully in the [activity user guide](https://github.com/dotnet/corefx/blob/master/src/System.Diagnostics.DiagnosticSource/src/ActivityUserGuide.md), but is described
here as well.
documented fully in the
[activity user guide](https://github.com/dotnet/corefx/blob/master/src/System.Diagnostics.DiagnosticSource/src/ActivityUserGuide.md),
Copy link
Member

Choose a reason for hiding this comment

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

this is the old link, which doesn't contain the current recommendation of using ActivitySource.

https://docs.microsoft.com/dotnet/core/diagnostics/distributed-tracing#getting-started-with-tracing is the right link.

@cijothomas
Copy link
Member

@plmaheu Will you be updating the link to the correct one ?

@cijothomas
Copy link
Member

Closing this PR.
The actual .NET docs are being refactored and updated now. Will point to the correct doc once that docs achieve some stability.

@cijothomas cijothomas closed this Mar 16, 2021
@reyang
Copy link
Member

reyang commented Mar 16, 2021

dotnet/docs#23313

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