Closed henriquesposito closed 2 years ago
Merging #61 (d0264c7) into main (2329515) will decrease coverage by
2.75%
. The diff coverage is58.46%
.:exclamation: Current head d0264c7 differs from pull request most recent head 9dab641. Consider uploading reports for the commit 9dab641 to get more accurate results
@@ Coverage Diff @@
## main #61 +/- ##
==========================================
- Coverage 91.50% 88.75% -2.76%
==========================================
Files 11 12 +1
Lines 765 818 +53
==========================================
+ Hits 700 726 +26
- Misses 65 92 +27
Impacted Files | Coverage Δ | |
---|---|---|
R/report.R | 41.30% <41.30%> (ø) |
|
R/contract.R | 100.00% <100.00%> (ø) |
|
R/expand.R | 100.00% <100.00%> (ø) |
|
R/logical.R | 100.00% <100.00%> (ø) |
Should we keep the article in a separate branch for now? If we are to include it here we should also, perhaps, mentions it in the read.me and/or in the website, as well as change the JSS format to another one. What do you think @jhollway and @jaeltan ?
Description
These patch changes to
{messydates}
include the addition of themreport()
function to properly report on data with 'mdate' variables.Package
battles
data included in packageFunctions
mreport()
function to properly report on data containing 'mdate' variablesexpand()
functionChecklist: