CRAN submission comments
jhrcook opened this issue · comments
The following are the comments from CRAN after the first submission attempt:
- Please omit the redundant "in R" from the title.
- Please add \value to .Rd files regarding exported methods and explain the functions results in the documentation. Please write about the structure of the output (class) and also what the output means. (If a function does not return a value, please document that too, e.g. \value{No return value, called for side effects} or similar)
- \dontrun{} should only be used if the example really cannot be executed (e.g. because of missing additional software, missing API keys, ...) by the user. That's why wrapping examples in \dontrun{} adds the comment ("# Not run:") as a warning for the user. Does not seem necessary. Please replace \dontrun with \donttest.
- Please do not modifiy the .GlobalEnv. This is not allowed by the CRAN policies.
These are being addressed in the cran-submission-comments
branch.
My comments:
- Easy change to documentation.
- Easy changes to documentation.
- Change to documentation that may be a bit tricky to resolve.
- I do not know how to get around this one, but I have reached out to the maintainer of 'ProjectTemplate' for advice.
Kenton White, the maintainer of 'ProjectTemplate', replied to my query:
We were grandfathered in before CRAN started the policy of not modifying the .GlobalEnv. I have had challenges with CRAN and have found that if I make a well reasoned argument why it must be done this way they can be persuaded. Sometimes they have a better suggestion than what I was thinking!
I will take his advice and add some comments to the cran-comments.md
file.