Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs(readme): fix @opentelemetry/instrumentation-http link #1973

Merged
merged 2 commits into from
Mar 1, 2021

Conversation

hongbo-miao
Copy link
Contributor

@hongbo-miao hongbo-miao commented Feb 26, 2021

  1. @opentelemetry/instrumentation-http was pointing to old plugin link.
  2. We actually don't have @opentelemetry/instrumentation-https because new @opentelemetry/instrumentation-http will handle both HTTP and HTTPs, so remove that not used link.

@codecov
Copy link

codecov bot commented Feb 26, 2021

Codecov Report

Merging #1973 (e63d7de) into main (67f7e70) will decrease coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1973      +/-   ##
==========================================
- Coverage   92.90%   92.78%   -0.13%     
==========================================
  Files         152      147       -5     
  Lines        5922     5698     -224     
  Branches     1245     1193      -52     
==========================================
- Hits         5502     5287     -215     
+ Misses        420      411       -9     
Impacted Files Coverage Δ
packages/opentelemetry-web/src/utils.ts
...ntelemetry-web/src/enums/PerformanceTimingNames.ts
...ackages/opentelemetry-web/src/WebTracerProvider.ts
...kages/opentelemetry-web/src/StackContextManager.ts
packages/opentelemetry-web/src/types.ts

@dyladan dyladan added the document Documentation-related label Mar 1, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
document Documentation-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants