Conversation
R CMD check only allows calls to person(), c(person(), ...).
Found the following (possibly) invalid URLs: URL: https://mac.r-project.org/openmp (moved to https://mac.r-project.org/openmp/) From: NEWS.md Status: 301 Message: Moved Permanently
Found the following (possibly) invalid URLs: URL: https://www.rdocumentation.org/packages/data.table/functions/setkey From: inst/doc/datatable-faq.html Status: 502 Message: Bad Gateway
Found the following (possibly) invalid URLs: URL: https://www.rdocumentation.org/trends From: README.md Status: 502 Message: Bad Gateway
* checking package subdirectories ... NOTE
Problems with news in ‘NEWS.md’:
Cannot extract version info from the following section titles:
NOTES
* checking Rd line widths ... NOTE
Rd file 'nafill.Rd':
\usage lines wider than 90 characters:
setnafill(x, type=c("const", "locf", "nocb"), fill=NA, nan=NA, cols=seq_along(x),
limit=Inf)
R CMD check now requires \usage{} entries for all exported functions:
Exported functions without usage information:
melt dcast
Member
|
thanks Ivan. |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Authors@Ris now required to only have calls topersonandc. We could also prepare a patch for R to allowNULLin thec(...)call once again.https://www.rdocumentation.orgreturning 502NEWS.mdhad one heading with a wrong level, which confusednews()\usageline length (so that the line fits into the PDF page)--as-crannow includes_R_CHECK_CODOC_FUNCTIONS_MISSING_FROM_USAGES_=NA, which enables a test intools::codoc()requiring a\usage{}entry for every exported object. For now, this is not even a NOTE (only INFO) and shouldn't affect CRAN submission. These were easy to add for themeltanddcastgenerics, but not for:=orlet. We could:rlang...`:=`(...)andlet(...)saying something like "please don't do this". This would also force us to document the...argument.:=to(LHS, RHS)and add usage entries forLHS := RHSandlet(...)(with a comment that this only works inside a data.table operation). Document...as a sequence of taggedLHS = RHSpairs. This isn't entirely right because:=also accepts a sequence of tagged arguments.