Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Parse dropped timeseries from Create Timeseries Response #276

Conversation

james-bebbington
Copy link
Contributor

See the test case for an example of a partial success response

@james-bebbington james-bebbington force-pushed the parse-dropped-metrics branch 2 times, most recently from 20db006 to 55edfb9 Compare August 25, 2020 12:26
metrics_batcher_test.go Outdated Show resolved Hide resolved
metrics_batcher.go Show resolved Hide resolved
metrics_batcher_test.go Show resolved Hide resolved
@james-bebbington james-bebbington force-pushed the parse-dropped-metrics branch 2 times, most recently from 5350e52 to 195160c Compare August 25, 2020 23:54
@codecov-commenter
Copy link

Codecov Report

Merging #276 into master will increase coverage by 0.34%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #276      +/-   ##
==========================================
+ Coverage   72.37%   72.72%   +0.34%     
==========================================
  Files          17       17              
  Lines        1698     1712      +14     
==========================================
+ Hits         1229     1245      +16     
+ Misses        392      391       -1     
+ Partials       77       76       -1     
Impacted Files Coverage Δ
metrics_batcher.go 88.65% <100.00%> (+4.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 cde3c10...195160c. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants