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

add Imports and Depends to DESCRIPTION #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wenjie2wang
Copy link

Nice package.

This PR suggests a minor modification to the DESCRIPTION file.

I think it would be great to add Imports and Depends entries to the DESCRIPTION so that people can easily know the package is light-weighted and does not require many other packages installed from CRAN. I had to check the NAMESPACE and found out it only require utils package (and base R of course).

As for the Depends, I just checked time of the first commit for this package (Sept. 2015) and put in the latest version of R at that time. You may relax the requirement. But it would be good to have one. After all the package itself is about "versions".

Feel free to decline this PR if you find it unnecessary.

@goldingn
Copy link
Owner

Thanks! I'm glad you like the package.

This looks good. I'll first check the minimum R version that works, since I don't want to prevent people on outdated (but still working) R installation from being able to install the package.

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