-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update description #53
Conversation
Package description based on what I wrote for https://ropensci.org/blog/2019/01/22/waterinfo-tidal-eel/, with more focus on scope of environmental variables.
Is more consistent with other R package websites. I left CONTRIBUTING in for now 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the suggestions! I would rather prefer to keep some elements asa they were before. Can you update the website pages itself?
We'll need to fix the unit test issue, but these has nothing to do with your additions, so you can ignore these and we'll do a fix in a separate branch.
Package description based on what I wrote for https://ropensci.org/blog/2019/01/22/waterinfo-tidal-eel/, with more focus on scope of environmental variables.
Is more consistent with other R package websites. I left CONTRIBUTING in for now 😄
e026e79
to
686400e
Compare
As per request @stijnvanhoey
We still need to update the website itself before merging. |
Site generated. 👌 |
Update description of package in
README
andDESCRIPTION
(in part based on eel blog post). It now states:The PR also includes:
To get started, see:
(cf. bioRad or rgbif) inREADME
with links to function reference and articles (until now, there was no link to website)_pkgdown.yml
so these sections are calledReference
andArticles
in that order. The website has not been regenerated yet.