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

Deprecate and add warning to remove AtomicFieldOffsets, unnecessary public func #2445

Merged
merged 2 commits into from Dec 15, 2021

Conversation

bogdandrutu
Copy link
Member

Signed-off-by: Bogdan Drutu bogdandrutu@gmail.com

…ublic func

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
@codecov
Copy link

codecov bot commented Dec 13, 2021

Codecov Report

Merging #2445 (039e1fe) into main (8f4a477) will decrease coverage by 0.0%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #2445     +/-   ##
=======================================
- Coverage   76.3%   76.3%   -0.1%     
=======================================
  Files        173     173             
  Lines      12010   12010             
=======================================
- Hits        9170    9165      -5     
- Misses      2595    2600      +5     
  Partials     245     245             
Impacted Files Coverage Δ
sdk/metric/atomicfields.go 0.0% <ø> (-100.0%) ⬇️

@bogdandrutu bogdandrutu added the area:metrics Part of OpenTelemetry Metrics label Dec 14, 2021
@MrAlias MrAlias merged commit 4702f6f into open-telemetry:main Dec 15, 2021
@MrAlias MrAlias mentioned this pull request Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:metrics Part of OpenTelemetry Metrics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants