-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
thanks for this @llrs having a look |
vignettes/jaod.Rmd
Outdated
) | ||
``` | ||
|
||
Vignettes are long form documentation commonly included in packages. Because they are part of the distribution of the package, they need to be as compact as possible. The `html_vignette` output type provides a custom style sheet (and tweaks some options) to ensure that the resulting html is as small as possible. The `html_vignette` format: |
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.
Can you remove this paragraph and bulleted list? We usually don't introduce the vignette format in vignettes.
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.
Oh, sorry I forgot to remove it
Vignettes are long form documentation commonly included in packages. Because they are part of the distribution of the package, they need to be as compact as possible. The `html_vignette` output type provides a custom style sheet (and tweaks some options) to ensure that the resulting html is as small as possible. The `html_vignette` format: |
Looks like you need to merge in changes from master before we can merge here |
Merge branch 'master' of https://github.com/ropensci/jaod # Conflicts: # DESCRIPTION
Codecov Report
@@ Coverage Diff @@
## master #7 +/- ##
=======================================
Coverage 94.82% 94.82%
=======================================
Files 6 6
Lines 58 58
=======================================
Hits 55 55
Misses 3 3 Continue to review full report at Codecov.
|
Following the README.Rmd I created this PR to add a vignette (related to #2).
While creating the vignette I found is a lack of functions to use the data retrieved.
Also having a summary of the results obtained printed on the console when performing a query could come handy.