Summary:
- CRAN prohibits by-default writing to the user's home filespace, the package directory, or the working directory (getwd()) in examples, vignettes, or tests.
Feedback references:
Initial code review findings:
- In
R/report.R, the default for file outputs is already tempfile(), but please confirm/extend this pattern in all user-writable function parameters.
- All examples, vignettes, and test code that writes files should explicitly use
tempdir() (never package root, never working directory unless set within a temp directory).
- The vignette file
deploy-local-datashield-server-with-opal.Rmd appears not to write output, but all future/output code must use tempdir() as root for all paths.
Checklist:
Summary:
Feedback references:
Initial code review findings:
R/report.R, the default for file outputs is alreadytempfile(), but please confirm/extend this pattern in all user-writable function parameters.tempdir()(never package root, never working directory unless set within a temp directory).deploy-local-datashield-server-with-opal.Rmdappears not to write output, but all future/output code must usetempdir()as root for all paths.Checklist:
tempdir()ortempfile()