Skip to content

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