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

Add init and finish hook for knitr::knit() #2223

Open
cderv opened this issue Feb 10, 2023 · 1 comment
Open

Add init and finish hook for knitr::knit() #2223

cderv opened this issue Feb 10, 2023 · 1 comment
Labels
feature Feature requests

Comments

@cderv
Copy link
Collaborator

cderv commented Feb 10, 2023

This idea came up with discussion for #2219 in r-lib/rlang#1551

At first it was something that could have been needed as rlang needed to set thinks at start of knitting, and reset after knitting. Solution was done another way (r-lib/rlang@6b3cbe9)

However, the idea is interesting to have those hooks available for other to extend knitr. So leaving this issue open to keep track of the idea, but we can wait for another real use case.

@cderv cderv added the feature Feature requests label Feb 10, 2023
@yihui
Copy link
Owner

yihui commented Feb 13, 2023

I'm okay with adding these hooks.

res = process_file(text, output)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature requests
Projects
None yet
Development

No branches or pull requests

2 participants