Enable use of pandoc --file-scope for input files originating from multiple Rmds #1837
Conversation
This file contains 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
…rom multiple Rmds. Caller must provide an md_file_splitter funciton used to break apart a markdown file to be passed to pandoc into multiple files. This in turn enables the use of pandoc's --file-scope, which in turn resolves duplicate numeric footnote id's across files. In the short term this will be used by bookdown to enable correct handling of duplicate numbered footnotes across chapters.
jjallaire
added a commit
to rstudio/bookdown
that referenced
this pull request
Jun 4, 2020
…meter. Numeric footnotes duplicated across chapters are now automatically renumbered. This is done by passing the `--file-scope` argument to pandoc (and having it operate on split out individual chapters of the target .md file rather than a combined file). This behavior can be toggled off by setting `pandoc_file_scope` to `false` in `_bookdown.yml`. See also rstudio/rmarkdown#1837 (which this PR depends on)
jonathan-g
added a commit
to jonathan-g/rmarkdown
that referenced
this pull request
Jul 7, 2020
* rstudio/master: (61 commits) make metadata available before running pre_knit (rstudio#1855) fix rstudio#1815: apply the lua filters pagebreak.lua and latex-div.lua to beamer_presentation require the data-latex attribute on fenced Div's again: rstudio#1779 (comment) start the next version CRAN release v2.3 fold any code blocks with the class `foldable` for html_document output (rstudio#1835) fix rstudio#1828: add aria-hidden = "true" to empty <a> tags in highlighted code blocks generated by Pandoc to improve screen reader accessibility (rstudio#1833) add a news item for rstudio#1832 roxygenize no need to turn on --file-scope or actually write the split content into files if the split content is of length < 2 eliminate renumber_footnotes option only test two pandoc versions (devel and RStudio version) and upgrade default to 2.7.3 (rstudio#1846) close rstudio#1838: test more R versions on Travis (rstudio#1845) renumber_footnotes output format option add the <div class="kable-table"> only when the output format is HTML, otherwise the div will be converted to a LaTeX environment, leading to the bug https://stackoverflow.com/q/62340425/559676 change name of file_scope argument to references_scope re-roxygenize Add `publish_site()` function for "one-button" publishing of R Markdown websites. Enable use of pandoc --file-scope for input files originating from multiple Rmds (rstudio#1837) Added missing lang attribute to ioslides_presentation template (rstudio#1841) ...
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Caller must provide a
file_scopefunction used to break apart a markdown file to be passed to pandoc into multiple files. This in turn enables the use of pandoc's--file-scope(https://pandoc.org/MANUAL.html#option--file-scope), which in turn resolves duplicate numeric footnote id's across files.In the short term this will be used by bookdown to enable correct handling of duplicate numbered footnotes across chapters.
Note that if
render()handled multiple input files by passing them all to pandoc, we could makefile_scopea boolean or a function (where the boolean would result in just passing--file-scopewithout any splitting). However, for multiple inputsrender()calls itself recursively (meaning that pandoc always gets a single Rmd anyway, so--file-scopewould have no meaning).