Skip to content

Changing install packages to CRAN Library #57

Open
GraysenF1 wants to merge 2 commits intocrkn-rcdr:mainfrom
GraysenF1:fixing-install-packages
Open

Changing install packages to CRAN Library #57
GraysenF1 wants to merge 2 commits intocrkn-rcdr:mainfrom
GraysenF1:fixing-install-packages

Conversation

@GraysenF1
Copy link

#18 fixed

Helps explain what this folder does to someone who pokes around the repository without running the code first.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The rcrossref and geonames package installation should not include "ropensci/" as this was part of the path on GitHub. There is also a typo: in line 27, add an "r" since the package is called rcrossref.
Confirmed that these are current packages by looking at CRAN documentation for rcrossref and geonames

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The rcrossref and geonames package installation should not include "ropensci/" as this was part of the path on GitHub. There is also a typo: in line 27, add an "r" since the package is called rcrossref.
Confirmed that these are current packages by looking at CRAN documentation for rcrossref and geonames

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change the word file to folder, since it is the folder that is empty.

Adding a description of each of the data files generated by the Rorcid_Crossref_Authors.R file would add value to this README.

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