Skip to content
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

Allow people to use the {pandoc} R package with {sandpaper} #465

Closed
zkamvar opened this issue May 18, 2023 · 0 comments · Fixed by #466
Closed

Allow people to use the {pandoc} R package with {sandpaper} #465

zkamvar opened this issue May 18, 2023 · 0 comments · Fixed by #466
Labels
bug Something isn't working

Comments

@zkamvar
Copy link
Contributor

zkamvar commented May 18, 2023

The {pandoc} R package is a wonderful addition that's allowed me to test different versions of pandoc... but I've run into a problem while testing #454 where I've been unable to activate the correct version of pandoc. It turns out that, because I run the pandoc conversions through callr, that uses the default system pandoc:

callr::r(function(...) rmarkdown::pandoc_convert(...), args = args,
show = !quiet, spinner = sho)

I would love to be able to take {pandoc} as a dependency, but the problem is that it requires a GitHub PAT to be set up to connect to GitHub to download releases.

I might be able to get around this by passing the results from rmarkdown::find_pandoc() into the callr function so that it can set the right pandoc version before running.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant