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

pkgcheck results - ropensci #57

Closed
github-actions bot opened this issue Aug 24, 2022 · 7 comments
Closed

pkgcheck results - ropensci #57

github-actions bot opened this issue Aug 24, 2022 · 7 comments

Comments

@github-actions
Copy link

Checks for fwildclusterboot (v0.11)

git hash: 00c38b8d

  • ✔️ Package is already on CRAN.
  • ✔️ has a 'codemeta.json' file.
  • ✔️ has a 'contributing' file.
  • ✖️ The following functions have no documented return values: [boot_ssc, pval, teststat]
  • ✔️ uses 'roxygen2'.
  • ✔️ 'DESCRIPTION' has a URL field.
  • ✔️ 'DESCRIPTION' has a BugReports field.
  • ✔️ Package has at least one HTML vignette
  • ✔️ All functions have examples.
  • ✖️ Function names are duplicated in other packages
  • ✔️ Package has continuous integration checks.
  • ✖️ Package coverage is 58.6% (should be at least 75%).
  • ✔️ R CMD check found no errors.
  • ✔️ R CMD check found no warnings.

Important: All failing checks above must be addressed prior to proceeding

Package License: GPL-3

@s3alfisc
Copy link
Owner

  • cmd check fails because pkgcheck does not find the JULIA_BINDIR environment flag - the workflow of course does not install Julia, WildBootTests.jl etc, and it fails because I failed to add the clause that exludes the test if Julia is not found. I have changed this in a recent commit
  • coverage is still low because I am excluding all Julia tests (except the one that fails)
  • the concerned duplicated function / method names are pval and teststat - I like these function names and think it is in fact a benefit that they are used in other packages
  • the only remaining library() calls are in vignettes, readme and news files. Should I get rid of all of these and replace with requireNamespace()?
  • when running pkgcheck locally, I only find one line with more than 80 characters - I have kept this line because otherwise, the argument checks via the dreamerr package would fail (dreamerr does not work nicely with linebreaks)

@github-actions
Copy link
Author

Checks for fwildclusterboot (v0.11)

git hash: 2fba1998

  • ✔️ Package is already on CRAN.
  • ✔️ has a 'codemeta.json' file.
  • ✔️ has a 'contributing' file.
  • ✔️ uses 'roxygen2'.
  • ✔️ 'DESCRIPTION' has a URL field.
  • ✔️ 'DESCRIPTION' has a BugReports field.
  • ✔️ Package has at least one HTML vignette
  • ✔️ All functions have examples.
  • ✖️ Function names are duplicated in other packages
  • ✔️ Package has continuous integration checks.
  • ✖️ Package coverage is 60.7% (should be at least 75%).
  • ✔️ R CMD check found no errors.
  • ✔️ R CMD check found no warnings.

Important: All failing checks above must be addressed prior to proceeding

Package License: GPL-3

@assignUser
Copy link

assignUser commented Aug 27, 2022

@s3alfisc You can change your pkgcheck action file to install additional system dependencies like Julia (e.g. via an additional step using https://github.com/julia-actions/setup-julia)
We have an open issue to add documentation about that (with a small example on how to do it: ropensci-review-tools/pkgcheck-action#23 (comment)

@s3alfisc
Copy link
Owner

Hi @assignUser, thanks for your feedback! I had in fact done this the other day but then reverted the commit as the unit tests run for a while, and I first wanted to make sure that pkgcheck() runs without me fiddling with the yaml file. I'll try to keep an eye on the issue and would be happy to eventually add a Julia example to the documentation.

@github-actions
Copy link
Author

Checks for fwildclusterboot (v0.11.1)

git hash: 095e62ff

  • ✔️ Package is already on CRAN.
  • ✔️ has a 'codemeta.json' file.
  • ✔️ has a 'contributing' file.
  • ✔️ uses 'roxygen2'.
  • ✔️ 'DESCRIPTION' has a URL field.
  • ✔️ 'DESCRIPTION' has a BugReports field.
  • ✔️ Package has at least one HTML vignette
  • ✔️ All functions have examples.
  • ✖️ Function names are duplicated in other packages
  • ✔️ Package has continuous integration checks.
  • ✖️ Package coverage is 59.6% (should be at least 75%).
  • ✔️ R CMD check found no errors.
  • ✔️ R CMD check found no warnings.

Important: All failing checks above must be addressed prior to proceeding

Package License: GPL-3

@github-actions
Copy link
Author

Checks for fwildclusterboot (v0.11.1)

git hash: 2f1460d5

  • ✔️ Package is already on CRAN.
  • ✔️ has a 'codemeta.json' file.
  • ✔️ has a 'contributing' file.
  • ✔️ uses 'roxygen2'.
  • ✔️ 'DESCRIPTION' has a URL field.
  • ✔️ 'DESCRIPTION' has a BugReports field.
  • ✔️ Package has at least one HTML vignette
  • ✔️ All functions have examples.
  • ✖️ Function names are duplicated in other packages
  • ✔️ Package has continuous integration checks.
  • ✖️ Package coverage is 59.7% (should be at least 75%).
  • ✔️ R CMD check found no errors.
  • ✔️ R CMD check found no warnings.

Important: All failing checks above must be addressed prior to proceeding

Package License: GPL-3

@github-actions
Copy link
Author

Checks for fwildclusterboot (v0.11.1)

git hash: 5500287f

  • ✔️ Package is already on CRAN.
  • ✔️ has a 'codemeta.json' file.
  • ✔️ has a 'contributing' file.
  • ✔️ uses 'roxygen2'.
  • ✔️ 'DESCRIPTION' has a URL field.
  • ✔️ 'DESCRIPTION' has a BugReports field.
  • ✔️ Package has at least one HTML vignette
  • ✔️ All functions have examples.
  • ✖️ Function names are duplicated in other packages
  • ✔️ Package has continuous integration checks.
  • ✖️ Package coverage is 59.7% (should be at least 75%).
  • ✔️ R CMD check found no errors.
  • ✔️ R CMD check found no warnings.

Important: All failing checks above must be addressed prior to proceeding

Package License: GPL-3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants