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

codyn fails to import 'stats' package methods #69

Closed
mbjones opened this issue Apr 26, 2016 · 1 comment
Closed

codyn fails to import 'stats' package methods #69

mbjones opened this issue Apr 26, 2016 · 1 comment
Assignees
Labels
Milestone

Comments

@mbjones
Copy link
Member

mbjones commented Apr 26, 2016

Newer versions of R require that 'stats' methods be explicitly imported rather than being assumed as available. Add 'importFrom' throughout the R code, and rebuild the NAMESPACE to support those.

In the future, ensure there is an ROxygen import statement for every library in use.

@mbjones mbjones self-assigned this Apr 26, 2016
@mbjones mbjones added the bug label Apr 26, 2016
@mbjones mbjones added this to the 1.1.0 milestone Apr 26, 2016
mbjones added a commit that referenced this issue Apr 26, 2016
These have been added in ROxygen importFrom statements, and the NAMESPACE was rebuilt.  Closes issue #69.
@mbjones
Copy link
Member Author

mbjones commented Apr 26, 2016

@laurenmh and @aammd -- note that for future additions we need to properly import functions -- you need lines like this for every external function that you use:

#' @importFrom stats aggregate as.formula sd

Alternatively, if you prefer to import the whole package, you can use:

#' @import stats

@mbjones mbjones closed this as completed Apr 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant