Skip to content
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

BioC 3.8 - BiocCheck #56

Closed
2 tasks done
nhejazi opened this issue Nov 10, 2018 · 0 comments · Fixed by #57
Closed
2 tasks done

BioC 3.8 - BiocCheck #56

nhejazi opened this issue Nov 10, 2018 · 0 comments · Fixed by #57

Comments

@nhejazi
Copy link
Owner

nhejazi commented Nov 10, 2018

A current run of BiocCheck::BiocCheck(".") returns a few interesting NOTEs, given below

  • "Avoid sapply(); use vapply() found in files:"
  • "Use accessors; don't access S4 class slots via '@' in examples/vignettes."
Summary:
ERROR count: 0
WARNING count: 3
NOTE count: 6
For detailed information about these checks, see the BiocCheck
vignette, available at
https://bioconductor.org/packages/3.7/bioc/vignettes/BiocCheck/inst/doc/BiocCheck.html#interpreting-bioccheck-output
$error
character(0)

$warning
[1] "y of x.y.z version should be even in release"                                               
[2] "Update R version dependency from 3.4 to 3.5."                                               
[3] "The following files are over 5MB in size: '.git/objects/pack/pack-fa09b8fab3b8aef3e71f2ecc0640cfe4994fb0cd.pack'"

$note
[1] "Avoid sapply(); use vapply() found in files:"                                               
[2] "Use accessors; don't access S4 class slots via '@' in examples/vignettes."                  
[3] "Consider adding runnable examples to the following man pages which document exported objects:"
[4] "Consider shorter lines; 1 lines (0%) are > 80 characters long."                             
[5] "Consider multiples of 4 spaces for line indents, 264 lines(18%) are not."                   
[6] "Cannot determine whether maintainer is subscribed to the bioc-devel\nmailing list (requires admin credentials).  Subscribe here:\nhttps://stat.ethz.ch/mailman/listinfo/bioc-devel"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant