-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Update tracing docs and add note to the @distributed_trace
decorator that it's not intended for end users
#37478
Conversation
API change check API changes are not detected in this pull request. |
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.
Thanks for this! Seems our "Verify Readmes" check isn't happy with the lack of "Key concepts", "Examples", and "Next steps" sections in the README file. I think we can either squeeze the section headings in, or perhaps just add the README path here. Other core libraries don't necessarily conform to these headings anyway.
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.
Identity CI failures were addressed in main
and can be overridden.
thank you @pvaneck ! |
25334c8
to
823483c
Compare
823483c
to
d47a4b0
Compare
Fixes #37342
Updates tracing plugin docs to use AzMon OTel distro instead of exported.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines