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

Move some content from correctness_test.go to utils.go #1497

Merged
merged 4 commits into from
Aug 5, 2020
Merged

Move some content from correctness_test.go to utils.go #1497

merged 4 commits into from
Aug 5, 2020

Conversation

pmcollins
Copy link
Member

This change makes these functions/types available from the metrics package, where they will be needed to address issue #652.

This change makes these functions/types available from the metrics
package, where they will be needed to address issue #652.
@codecov
Copy link

codecov bot commented Aug 5, 2020

Codecov Report

Merging #1497 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1497      +/-   ##
==========================================
+ Coverage   90.83%   90.84%   +0.01%     
==========================================
  Files         234      234              
  Lines       16477    16477              
==========================================
+ Hits        14967    14969       +2     
+ Misses       1082     1081       -1     
+ Partials      428      427       -1     
Impacted Files Coverage Δ
translator/internaldata/resource_to_oc.go 86.04% <0.00%> (+2.32%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87b1c14...2c79b4a. Read the comment docs.

@bogdandrutu bogdandrutu self-assigned this Aug 5, 2020
@bogdandrutu bogdandrutu merged commit 56a22ad into open-telemetry:master Aug 5, 2020
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
Troels51 pushed a commit to Troels51/opentelemetry-collector that referenced this pull request Jul 5, 2024
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.

2 participants