Closed m-muecke closed 5 months ago
Attention: 6 lines
in your changes are missing coverage. Please review.
Comparison is base (
8f9577f
) 69.50% compared to head (f43b902
) 69.44%.:exclamation: Current head f43b902 differs from pull request most recent head 966176b. Consider uploading reports for the commit 966176b to get more accurate results
Files | Patch % | Lines |
---|---|---|
R/convert-construct-utils.R | 33.33% | 4 Missing :warning: |
R/methods.R | 60.00% | 2 Missing :warning: |
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
thx!
stopifnot(inherits(...))
forcheckmate::assert_class(...)
stopifnot(is.numeric(x))
forcheckmate::assert_numeric(x)
round_resolution
and add tests for it.5
pattern for0.5
for consistencyanyNA
(more efficient)expect_length
,expect_lt
andexpect_gt
paste(..., collapse = ", ")
withtoString(x)
anyDuplicated(.)
instead of implicitfixed = TRUE
withgrep
where relevant (more efficient)nzchar
for checking empty strings (more efficient)