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.

Sign up for GitHub

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

Jump to bottom

LazyData - remove from default fields #104

Closed
kapsner opened this issue Mar 29, 2021 · 0 comments · Fixed by #105
Closed

LazyData - remove from default fields #104

kapsner opened this issue Mar 29, 2021 · 0 comments · Fixed by #105
Labels
bug an unexpected problem or unintended behavior good first issue ❤️ good issue for first-time contributors help wanted ❤️ we'd love your help!

Comments

@kapsner
Copy link

kapsner commented Mar 29, 2021 •
edited
Loading

CRAN checks in development version "using R Under development (unstable) (2021-03-26 r80126); using platform: x86_64-pc-linux-gnu (64-bit)" give a note, when LazyData: true is set in DESCRIPTION and no "data"-directory exists.

checking LazyData ... NOTE
'LazyData' is specified without a 'data' directory

However, true is the default value, when creating DESCRIPTION with the desc package.

I'd suggest to remove LazyData from default fields when creating a new DESCRIPTION file.

Thx.
@gaborcsardi gaborcsardi added bug an unexpected problem or unintended behavior good first issue ❤️ good issue for first-time contributors help wanted ❤️ we'd love your help! labels Mar 29, 2021
@kapsner kapsner changed the title LazyData - default "false" LazyData - remove from default fields Mar 29, 2021
@malcolmbarrett malcolmbarrett mentioned this issue Mar 29, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug an unexpected problem or unintended behavior good first issue ❤️ good issue for first-time contributors help wanted ❤️ we'd love your help!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

remove LazyData from !new malcolmbarrett/desc
2 participants
@gaborcsardi @kapsner