Skip to content
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

fix: fixup aws detector readme #2379

Merged
merged 1 commit into from
Jul 27, 2021

Conversation

legendecas
Copy link
Member

Which problem is this PR solving?

  • the code example in README is not valid.

@codecov
Copy link

codecov bot commented Jul 27, 2021

Codecov Report

Merging #2379 (e2ec6dd) into main (e089984) will increase coverage by 0.41%.
The diff coverage is n/a.

❗ Current head e2ec6dd differs from pull request most recent head 669cf63. Consider uploading reports for the commit 669cf63 to get more accurate results

@@            Coverage Diff             @@
##             main    #2379      +/-   ##
==========================================
+ Coverage   92.36%   92.77%   +0.41%     
==========================================
  Files         128      145      +17     
  Lines        4244     5221     +977     
  Branches      867     1070     +203     
==========================================
+ Hits         3920     4844     +924     
- Misses        324      377      +53     
Impacted Files Coverage Δ
...ource-detector-aws/src/detectors/AwsEc2Detector.ts 97.91% <ø> (ø)
...emetry-core/src/platform/node/RandomIdGenerator.ts 87.50% <0.00%> (-6.25%) ⬇️
.../opentelemetry-exporter-collector/src/transform.ts 88.69% <0.00%> (ø)
...ry-exporter-collector/src/CollectorExporterBase.ts 92.15% <0.00%> (ø)
...mentation-xml-http-request/src/enums/EventNames.ts 100.00% <0.00%> (ø)
...kages/opentelemetry-web/src/StackContextManager.ts 94.28% <0.00%> (ø)
...-instrumentation-fetch/src/enums/AttributeNames.ts 100.00% <0.00%> (ø)
...kages/opentelemetry-exporter-collector/src/util.ts 100.00% <0.00%> (ø)
...ackages/opentelemetry-web/src/WebTracerProvider.ts 100.00% <0.00%> (ø)
...emetry-instrumentation-xml-http-request/src/xhr.ts 97.58% <0.00%> (ø)
... and 9 more

@dyladan dyladan added the document Documentation-related label Jul 27, 2021
@dyladan dyladan merged commit 9c10bd7 into open-telemetry:main Jul 27, 2021
@legendecas legendecas deleted the fix/aws-detector branch July 28, 2021 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document Documentation-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants