-
Notifications
You must be signed in to change notification settings - Fork 773
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
[otlp] Add readme toc and clarify endpoint confusion #5385
[otlp] Add readme toc and clarify endpoint confusion #5385
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5385 +/- ##
==========================================
- Coverage 83.38% 83.31% -0.08%
==========================================
Files 297 278 -19
Lines 12531 11936 -595
==========================================
- Hits 10449 9944 -505
+ Misses 2082 1992 -90
Flags with carried forward coverage won't be shown. Click here to find out more.
|
src/OpenTelemetry.Exporter.OpenTelemetryProtocol/OtlpExporterOptions.cs
Outdated
Show resolved
Hide resolved
This PR was marked stale due to lack of activity and will be closed in 7 days. Commenting or Pushing will instruct the bot to automatically remove the label. This bot runs once per day. |
Fixes #
Design discussion issue #
Changes
Adds toc to otlp exporter Readme as it is getting bigger. Readme needs more organization, planning to do that separately. Added a note about endpoint confusion when using HttpProtobuf #5377
Merge requirement checklist
CHANGELOG.md
files updated for non-trivial changes