Skip to content

Put back LazyData, remove refinery from NAMESPACE #6

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

Merged
merged 1 commit into from
Jan 21, 2023

Conversation

dmurdoch
Copy link
Contributor

No description provided.

@sbgraves237
Copy link
Collaborator

Thanks. I think I will need to add "refinery" back into the NAMESPACE for a check we need but don't run on CRAN. However, there could be a hidden character someplace in what we read as "refinery". ??? Otherwise, I have another problem I don't understand. Spencer

@dmurdoch
Copy link
Contributor Author

You can have it in NAMESPACE if you save the object in R/sysdata.rda, or create it in code in some .R file in the R directory. It doesn't really make sense to use sysdata.rda, since "this is intended for system datasets that are not intended to be user-accessible via data" (WRE, section 1.1.5).

@sbgraves237 sbgraves237 merged commit 2fbdfb4 into JamesRamsay5:master Jan 21, 2023
@sbgraves237
Copy link
Collaborator

changes per Duncan Murdoch. Thanks. Spencer Graves

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

Successfully merging this pull request may close these issues.

2 participants