-
Notifications
You must be signed in to change notification settings - Fork 999
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
.NET manual: independent and new root docs #1867
.NET manual: independent and new root docs #1867
Conversation
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code example seems to be very complicated. I feel that we should encourage best practices in the doc which help to create a pit of success, please find my suggestion here https://github.com/open-telemetry/opentelemetry.io/pull/1867/files#r997371774
Co-authored-by: Severin Neumann <severin.neumann@altmuehlnet.de>
@reyang Sorry for not pinging, thought I had. Can you take another look to see if the changes are suitable? |
@reyang any chance to take another look? Looking to get through our open PRs this week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments inline. Thx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Co-authored-by: Patrice Chalin <chalin@users.noreply.github.com>
This fixes #1073 and brings the manual instrumentation bit up to par with JS.