-
Notifications
You must be signed in to change notification settings - Fork 8
Doc cleanup #625
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
base: dev
Are you sure you want to change the base?
Doc cleanup #625
Conversation
`downlit` fails to link these, maybe because we're not on CRAN yet.
vignette("backtesting", package="epipredict")
Inaccurate in that `before` isn't `.before` and `-` isn't `time_type`-aware. Misformatted in that `-` was interpreted as a bullet point.
9d1143b
to
8ac32e5
Compare
/preview-docs |
Check issues are probably unrelated, from #631. Will work on that. |
🚀 Deployed on https://67d8adaa245c833df3631f1d--epiprocess.netlify.app |
|
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.
I think roxygen adds those S3 method listings automatically. Any idea how to compress/remove them? I guess we could make them into |
Do the |
Checklist
Please:
PR).
brookslogan, nmdefries.
DESCRIPTION
. Always incrementthe patch version number (the third number), unless you are making a
release PR from dev to main, in which case increment the minor version
number (the second number).
(backwards-incompatible changes to the documented interface) are noted.
Collect the changes under the next release number (e.g. if you are on
1.7.2, then write your changes under the 1.8 heading).
process.
Change explanations for reviewer
See #563 checklist. Some other items of note are deferred to https://github.com/cmu-delphi/forecasting-team-meta/issues/29.
Magic GitHub syntax to mark associated Issue(s) as resolved when this is merged into the default branch