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

Move most readme content to a vignette #9

Closed
drmowinckels opened this issue Jun 2, 2020 · 4 comments
Closed

Move most readme content to a vignette #9

drmowinckels opened this issue Jun 2, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@drmowinckels
Copy link

This is an opinion, so it can absolutely be discussed.

in my opinion, the readme as a whole should be moved into a vignette folder and be compiled as such. This will also then run the vignette code during build checks, making sure that the vignette is actually executable (easiest way to do so it to use usethis::use_vignette() ).

The readme should be made concise and rather highlight what the package intention and core features are, while a full user tutorial is best placed in a vignette.

@samhforbes
Copy link
Owner

I really appreciate the benefit of your experience here @Athanasiamo. As you have suggested, the README has been simplified, with the expanded walkthrough moved to vignettes. In line with the other issues you've raised, the other tutorials are linked, community guidelines are flagged, and installation instructions are (hopefully) clear - more in line with the usethis templates. I have included badges to note the travis-CI and CRAN status as well.

@drmowinckels
Copy link
Author

This looks much neater in my opinion. Great you also added some badges indicating CRAN and CI status.

as a complete sidenote and something not related to the review, but something you might consider, is using pkgdown to generate a site for your package. https://pkgdown.r-lib.org/articles/pkgdown.html
Your package is set up correctly, meaning there is little you have to do to get it working. For people looking for your vignette for instance, it might be more convenient to see it hosted online (though github pages).

But, this is purely a suggestion. As a developer though, I can say that I have often linked to the online vignettes rather than the vignettes shipped with the package. Particularly for developer updates not yet available on CRAN.

@samhforbes
Copy link
Owner

This is a great suggestion, and again, thanks for the advice. I'm going to leave this open temporarily as a reminder to devote some time to getting a (pretty) page going

@samhforbes samhforbes added the enhancement New feature or request label Jun 8, 2020
@samhforbes
Copy link
Owner

It's pretty!
This is now up - currently at samforbes.me/PupillometryR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants