-
Notifications
You must be signed in to change notification settings - Fork 27
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
Remarks about the website #193
Comments
Thanks! The vignette title thing is because for some reason it doesn't use the vignette title if the vignette is already built - cause I use prebuilt vignette so they are not run on cran and on pkg installation. not sure how to fix that. Grouping sounds good, but what do you have in mind? |
Not sure exactly but e.g a group for the most useful function(s) ( |
sounds good, i guess occ and occ2df for most useful maybe |
Should I make a PR? |
sounds good |
Cool will do in 10 days or so |
Nearly two months later, she created the branch 🙈 Sorry for the delay, on it now. |
fix via #199 |
Cool that this package has a website!
The vignette doesn't have a title https://ropensci.github.io/spocc/articles/spocc_vignette.html It looks weird from the menu:

The reference is ordered alphabetically but it'd make sense to put the most important functions first and to group some functions (e.g. using the
roxygen2
trick https://ropensci.github.io/dev_guide/building.html#function-grouping)Am happy to make a PR for these two things if useful.
The text was updated successfully, but these errors were encountered: