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

Include metrics in release #2928

Merged
merged 2 commits into from
Apr 25, 2022
Merged

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented Apr 25, 2022

Metrics packages were left out of the release on friday because they were marked private

@dyladan dyladan requested a review from a team April 25, 2022 12:55
@codecov
Copy link

codecov bot commented Apr 25, 2022

Codecov Report

Merging #2928 (e0b7047) into main (4584c36) will increase coverage by 0.69%.
The diff coverage is n/a.

❗ Current head e0b7047 differs from pull request most recent head 5e6c3cf. Consider uploading reports for the commit 5e6c3cf to get more accurate results

@@            Coverage Diff             @@
##             main    #2928      +/-   ##
==========================================
+ Coverage   91.61%   92.31%   +0.69%     
==========================================
  Files         177      171       -6     
  Lines        5584     5346     -238     
  Branches     1208     1155      -53     
==========================================
- Hits         5116     4935     -181     
+ Misses        468      411      -57     
Impacted Files Coverage Δ
...-sdk-trace-web/src/enums/PerformanceTimingNames.ts
packages/opentelemetry-sdk-trace-web/src/utils.ts
...entelemetry-sdk-trace-web/src/WebTracerProvider.ts
packages/opentelemetry-sdk-trace-web/src/types.ts
...ckages/opentelemetry-sdk-trace-web/karma.worker.js
...telemetry-sdk-trace-web/src/StackContextManager.ts

@dyladan dyladan merged commit 28a177f into open-telemetry:main Apr 25, 2022
@dyladan dyladan deleted the publish-metrics branch April 25, 2022 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants