Closed SimonCMills closed 3 years ago
Ah, of course. Will do today
Merging #29 (28253c8) into main (620922c) will increase coverage by
2.40%
. The diff coverage is96.29%
.
@@ Coverage Diff @@
## main #29 +/- ##
==========================================
+ Coverage 26.20% 28.61% +2.40%
==========================================
Files 8 8
Lines 332 339 +7
==========================================
+ Hits 87 97 +10
+ Misses 245 242 -3
Impacted Files | Coverage Δ | |
---|---|---|
R/example_flocker_data.R | 96.87% <96.29%> (+12.87%) |
:arrow_up: |
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 620922c...28253c8. Read the comment docs.
Looks good to me, except that we are now failing unit tests for
test-make_flocker_data.R
, which were written based on the previous example data. If you can tweak those tests so that we're passing then feel free to merge this PR.