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

Example usage questions #65

Open
wjakethompson opened this issue Sep 11, 2023 · 2 comments
Open

Example usage questions #65

wjakethompson opened this issue Sep 11, 2023 · 2 comments

Comments

@wjakethompson
Copy link
Contributor

JOSS review criterion: Example usage (openjournals/joss-reviews#5835)

Example usage in the vignette currently points to external documents. These documents do a good job of illustrating example uses, but some were create many years ago (e.g., 2018). Is the functionality in those older documents still accurate, or have updates to the package in the last 5 years made those example obsolete? On the other hand, will future changes to EdSurvey potentially make current examples irrelevant.

Ideally, each of the example uses would be included as a vignette in the package (or an article that is only rendered on the pkgdown site if they are too big for CRAN). This would ensure that all the examples still work with the current version of the package, and would also ensure that future updates don't break the example (or that the examples are updated to reflect the updated package).

@pdbailey0
Copy link
Collaborator

pdbailey0 commented Sep 11, 2023

Thanks for pointing this out. EdSurvey was taking too long to build on the CRAN servers and was getting close to oversize, so we removed the vignettes to comply with a request from the CRAN maintainers to make it build at least two times faster.

The pkgdown site is an interesting alternative. We're actually moving our documentation to a book in package down. @wjakethompson If we updated the vignette to link there would that alleviate your concerns? Or are you saying it would be part of this site?

[Edited by PB to fix link formatting.]

@wjakethompson
Copy link
Contributor Author

I think in an ideal world it would be part of pkgdown site directly. If the example uses are part of the package itself, then they will be rebuilt whenever the package is updated, and throw and error if any of the changes to the package have broken the example use. This way, you would always know that the example uses were up to date with the current version of the package.

However, if EdSurvey is stable at this point and you aren't anticipating many updates in the future, then I think the bookdown site is sufficient.

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

No branches or pull requests

2 participants