Those were initially in this package to ensure that all docs were updated prior to generating the report. Those are, however, now the only part that requires actually loading the package. This is an issue in pre-submission enquiries which require srr checks, for which loading the package can take an hour or more. Removing the single roxygenise call here:
https://github.com/ropensci-review-tools/srr/blob/3dd673ac030219afbadbd6580646e19f2e6f86fa/R/report.R#L21-L26
will enable the full report to be generated without having to load the package at all, reducing time for check srr calls from potentially hours to a few seconds. That seems like a very worthy improvement.
Plan will be to expose an additional parameter, roxygenise = TRUE, and call it from roreviewapi with explicit FALSE value.
Those were initially in this package to ensure that all docs were updated prior to generating the report. Those are, however, now the only part that requires actually loading the package. This is an issue in pre-submission enquiries which require
srr
checks, for which loading the package can take an hour or more. Removing the singleroxygenise
call here: https://github.com/ropensci-review-tools/srr/blob/3dd673ac030219afbadbd6580646e19f2e6f86fa/R/report.R#L21-L26will enable the full report to be generated without having to load the package at all, reducing time for
check srr
calls from potentially hours to a few seconds. That seems like a very worthy improvement.Plan will be to expose an additional parameter,
roxygenise = TRUE
, and call it fromroreviewapi
with explicitFALSE
value.