-
-
Notifications
You must be signed in to change notification settings - Fork 970
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
tweaks intermediates #2098
tweaks intermediates #2098
Conversation
knitr 1.24+ and rmarkdown 2.0+ only accepts UTF-8
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.
Looks good. Thanks! Let's see if we would discover any problems when checking the reverse dependencies the next time we make the CRAN release.
@yihui I added the simplest for now: an option This would allow to generate Do you think it should be better to customize the all extension:
When ready to merge I'll add NEWS bullet |
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.
The current method you provided is enough (it doesn't matter if it is *.knit.md~
or *.md~
). No need to do further work. Thanks!
knitr 1.24+ and rmarkdown 2.0+ only accepts UTF-8 so
*.utf8.md
is not needed anymorecloses #2086