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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor fixes: roxygen and People.RData -> people.RData #25

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@zeehio
Copy link

zeehio commented Apr 29, 2016

In order to test master to write #24, I rebuilt the package, using a more recent roxygen version. Feel free to use it.

Additionally, I found an issue when loading data(people) that could be solved changing data/People.RData to data/people.RData.

This pull request addresses both issues. Feel free to ignore them if you don't like them 馃憤 .

Thanks for your time and effort on this package

zeehio added some commits Apr 29, 2016

Rename People.RData to people.RData.
On Linux, R-3.2.5, mdatools built from master, I
could not run `data(people)` (not found).

@zeehio zeehio changed the title Minor fixes Minor fixes: roxygen and People.RData -> people.RData Apr 29, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment