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

docs(README): update link to @opentelemetry/api package #2263

Merged
merged 3 commits into from
Jun 9, 2021

Conversation

nvenegas
Copy link
Contributor

@nvenegas nvenegas commented Jun 8, 2021

Which problem is this PR solving?

Short description of the changes

  • Update URL in README

Copy link
Member

@johnbley johnbley left a comment

Choose a reason for hiding this comment

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

Thanks for catching this!

Copy link
Member

@obecny obecny left a comment

Choose a reason for hiding this comment

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

lgtm

@dyladan dyladan added the document Documentation-related label Jun 9, 2021
@codecov
Copy link

codecov bot commented Jun 9, 2021

Codecov Report

Merging #2263 (cd43568) into main (4a1f2e5) will increase coverage by 0.42%.
The diff coverage is n/a.

❗ Current head cd43568 differs from pull request most recent head 394b14e. Consider uploading reports for the commit 394b14e to get more accurate results

@@            Coverage Diff             @@
##             main    #2263      +/-   ##
==========================================
+ Coverage   92.29%   92.72%   +0.42%     
==========================================
  Files         127      144      +17     
  Lines        4205     5179     +974     
  Branches      857     1060     +203     
==========================================
+ Hits         3881     4802     +921     
- Misses        324      377      +53     
Impacted Files Coverage Δ
packages/opentelemetry-web/src/utils.ts 94.80% <0.00%> (ø)
...ation-xml-http-request/src/enums/AttributeNames.ts 100.00% <0.00%> (ø)
...kages/opentelemetry-exporter-collector/src/util.ts 100.00% <0.00%> (ø)
...emetry-instrumentation-xml-http-request/src/xhr.ts 96.58% <0.00%> (ø)
...kages/opentelemetry-web/src/StackContextManager.ts 97.05% <0.00%> (ø)
...-instrumentation-fetch/src/enums/AttributeNames.ts 100.00% <0.00%> (ø)
...ackages/opentelemetry-web/src/WebTracerProvider.ts 100.00% <0.00%> (ø)
...lemetry-exporter-collector/src/transformMetrics.ts 95.71% <0.00%> (ø)
...ages/opentelemetry-exporter-collector/src/types.ts 100.00% <0.00%> (ø)
...es/opentelemetry-context-zone-peer-dep/src/util.ts 100.00% <0.00%> (ø)
... and 7 more

@dyladan dyladan merged commit 81c4bce into open-telemetry:main Jun 9, 2021
@nvenegas
Copy link
Contributor Author

Thanks, all!

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.

5 participants