Closed slwu89 closed 2 years ago
Merging #173 (7f84a8e) into dev (5e7bbec) will decrease coverage by
0.09%
. The diff coverage is100.00%
.:exclamation: Current head 7f84a8e differs from pull request most recent head a61d4ca. Consider uploading reports for the commit a61d4ca to get more accurate results
@@ Coverage Diff @@
## dev #173 +/- ##
==========================================
- Coverage 95.96% 95.86% -0.10%
==========================================
Files 35 34 -1
Lines 1758 1717 -41
==========================================
- Hits 1687 1646 -41
Misses 71 71
Impacted Files | Coverage Δ | |
---|---|---|
R/targeted_event.R | 90.47% <ø> (ø) |
|
inst/include/NumericVariable.h | 98.43% <100.00%> (-0.59%) |
:arrow_down: |
inst/include/RaggedVariable.h | 92.20% <100.00%> (-2.58%) |
:arrow_down: |
inst/include/vector_variables.h | 100.00% <100.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
I think the errors coming up on CRAN were related to R switching to HTML5 (https://blog.r-project.org/2022/04/08/enhancements-to-html-documentation/) for validation in the newest version (see this issue https://github.com/r-lib/roxygen2/issues/1290). I updated the Roxygen2 version needed and set roxygen2 to use markdown syntax for parsing Rd files. I also simplified the offending parts of TargetedEvent to use markdown style links to help clear things up. Everything now passes on r-hub builders (using https://r-hub.github.io/rhub/reference/check_for_cran.html)