-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Include CRAN error/warning/note where relevant #57
Comments
I started adding the standard response from CRAN team members for the last recipes (e.g.: the .GlobalEnv issue). When we are done with (most) recipes, I will go through and add them for the earlier recipes as well. Errors/warnings/notes from the automated checks are not implemented yet. The problem there is that I don't really see them when doing CRAN work as the packages are rejected before they reach me. I suppose those issues are more likely detected by package authors of the community. @jasdumas and I are discussing good ways, how they can contribute. We will also talk a bit about this in our time meeting next week. |
The other day was checking BiocChecks that implements the check for Bioconductor. ccd <- tools::CRAN_check_details()
library("dplyr")
f <- ccd |>
as.data.frame() |>
filter(nzchar(Output)) |>
distinct(Check, Output) |>
arrange(Check, Output) Which shows that currently there are 47 different checks failing (with different outputs). BTW, related to #55, the most common check message is about "installed package size" with one above 98Mb |
That's a great hint! I'll look into this and keep it in mind when adding some issues/NOTEs in the future. Thanks |
For cases that produce an error/warning/note, such as Formatting Software Names, it would be good if we could include the message produced so that the cookbook may be discovered if people do a web search based on the message.
This does introduce a bit of a maintenance issue, as the message may change over time, but I think it will increase the usefulness of the cookbook.
The text was updated successfully, but these errors were encountered: