Skip to content

Add OTLP exporter README#299

Merged
reyang merged 6 commits into
open-telemetry:masterfrom
nadiaciobanu:otlp-readme
Aug 26, 2020
Merged

Add OTLP exporter README#299
reyang merged 6 commits into
open-telemetry:masterfrom
nadiaciobanu:otlp-readme

Conversation

@nadiaciobanu
Copy link
Copy Markdown
Contributor

Add a README to the OTLP exporter directory to describe the exporter and how it can be used/configured.

@nadiaciobanu nadiaciobanu requested a review from a team August 25, 2020 23:23
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 25, 2020

Codecov Report

Merging #299 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #299   +/-   ##
=======================================
  Coverage   94.55%   94.55%           
=======================================
  Files         146      146           
  Lines        6610     6610           
=======================================
  Hits         6250     6250           
  Misses        360      360           

Comment thread exporters/otlp/README.md Outdated
Copy link
Copy Markdown
Member

@reyang reyang left a comment

Choose a reason for hiding this comment

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

LGTM.

Comment thread exporters/otlp/README.md
@reyang reyang added the pr:please-merge This PR is ready to be merged by a Maintainer (rebased, CI passed, has enough valid approvals, etc.) label Aug 26, 2020
@ZiweiZhao
Copy link
Copy Markdown

LGTM

Copy link
Copy Markdown

@IlyaKobelevskiy IlyaKobelevskiy left a comment

Choose a reason for hiding this comment

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

LGTM

@nadiaciobanu
Copy link
Copy Markdown
Contributor Author

@reyang Ready to merge!

@reyang reyang merged commit 7d4c181 into open-telemetry:master Aug 26, 2020
@nadiaciobanu nadiaciobanu deleted the otlp-readme branch August 26, 2020 21:21
GerHobbelt pushed a commit to GerHobbelt/opentelemetry-cpp that referenced this pull request Jun 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:please-merge This PR is ready to be merged by a Maintainer (rebased, CI passed, has enough valid approvals, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants