-
-
Notifications
You must be signed in to change notification settings - Fork 873
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
Improve knitr::spin
rendering with .Rmd
files
#2281
Comments
Documentation of the function says Lines 44 to 47 in 50e28da
Try using |
Thanks @cderv. I hadn't seen that in the documentation but pieced it together. Why not have the default be to use |
I'll let @yihui answer to that. |
I originally found about this function from the R Markdown Cookbook: https://bookdown.org/yihui/rmarkdown-cookbook/spin.html This example would suggest you get a pdf document out, but you get an html |
Thanks for the context, this is indeed confusing... @yihui as anything changed since we wrote the R Markdown cookbook ?
Otherwise, the cookbook should be updated 🤔 |
@kylebutts
We need to clarify in the book that the R script is supposed to be rendered via
@cderv That's probably a good idea. Anyway, the cookbook definitely needs a bit more clarification. |
This old thread has been automatically locked. If you think you have found something related to this, please open a new issue by following the issue guide (https://yihui.org/issue/), and link to this old issue if necessary. |
I recently discovered
knitr::spin
which I agree isknitr
's best hidden gem!The current version uses
knit2html
which (1) raises warning thatrmarkdown::render
should be used and (2) ignores theoutput
yaml option:knitr/R/spin.R
Lines 119 to 121 in 50e28da
A proposed solution that works with both
text
andhair
optionsWould you welcome a PR adding this change?
By filing an issue to this repo, I promise that
I understand that my issue may be closed if I don't fulfill my promises.
The text was updated successfully, but these errors were encountered: