-
Notifications
You must be signed in to change notification settings - Fork 5
Docs review #149
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
Merged
Merged
Docs review #149
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
70c8e70
Make assert_timeset_param() internal to match other asserts
capnrefsmmat e4aa0cd
Make do_request() internal
capnrefsmmat 007e14e
Mark several internal functions as internal
capnrefsmmat 340b035
Use roxygen's auto-generated package documentation
capnrefsmmat 0bda8d1
Organize the function reference by topic
capnrefsmmat c93ca02
Fix VignetteIndexEntry title
capnrefsmmat 86a8cd7
Ensure endpoint doc URLs are hyperlinked
capnrefsmmat 300cfcb
Correct covidcast() documentation
capnrefsmmat 5c57f34
Expand epirange() and timeset documentation
capnrefsmmat c8cf0c7
Make print() methods match the generic's default argument list
capnrefsmmat 97c5eb0
Softer opening for the Getting Started vignette
capnrefsmmat db848cf
Make styler happy
capnrefsmmat 8d6792d
Correct argument name in error message
capnrefsmmat 8e45a6a
Include DESCRIPTION link to documentation site
capnrefsmmat cf7c391
Make the autocomplete helper public and document it better
capnrefsmmat 67dd3d1
Update vignettes/epidatr.Rmd
dshemetov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,2 @@ | ||
| #' Delphi Epidata | ||
| #' | ||
| #' TODO | ||
| #' | ||
| #' @docType package | ||
| #' @name epidatr | ||
| NULL | ||
| #' @keywords internal | ||
| "_PACKAGE" |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
What a peculiar bit of boilerplate. I'll definitely be taking your word for it
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.
See https://roxygen2.r-lib.org/articles/rd-other.html#packages
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.
Was also curious, thanks!