-
Notifications
You must be signed in to change notification settings - Fork 0
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
Simplify workflow for updating documentation #14
Comments
I'm trying out the only- |
It's a fight between roxygen2, CHECK, and pkgdown to get something that looks okay and passes all checks. I've ended up with
|
seems pretty solved to me, especially with |
Yeah, I think @nmdefries pushed a solution to this to both epiprocess & epipredict. (I also drafted another approach in some miscellaneous merged epiprocess PR to take care of some CHECK/pkgdown stuff if we ever need to revisit the impl here.) |
Migrating from memory of meeting.
Updating just documentation for data objects currently requires:
Ideas for simplifying:
data(<obj name>)
(and could add one about needing epidatasets attached for listing its data sets with justdata()
).@export
and that's it.?
/help
should route to epidatasets documentation [since weImports:
it], assuming we have identical object naming. We lose the notes about not being able to usedata(<obj name>)
, but that's an dated paradigm anyway, so hopefully not a huge loss.epidatasets::
and/orlibrary(epidatasets)
in various downstream examples and vignettes.The text was updated successfully, but these errors were encountered: