-
-
Notifications
You must be signed in to change notification settings - Fork 872
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
Update to knitr breaks quarto? #2261
Comments
Indeed #2243 has broken quarto, but not related to #2248 directly. @yihui I am available to discuss this. This is the addition of a new argument in Either something to fix in Quarto, or we should take more into knitr how Quarto is using our internals functions in a specific way. |
Like I said last time, we should make as less use of We could move some R code from Quarto to knitr conditioned on |
It's great that Quarto is being tested against the dev version of knitr now. I wonder if there's anything else we could do for this issue. |
I think initially I left that open regarding quarto-dev/quarto-cli#5704 (comment) We fixed in quarto for 1.3 (patch) and next 1.4 but their could have been issue with user having oldest Quarto (bundled in RStudio for example) and updating knitr. This issue here so that we could see if something (workaround conditional to quarto version maybe ?) was needed. It seems there was no more report of this so maybe we're good. |
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 believe that one of the latest changes in
knitr
breaksquarto
(see this). I think the problem is that commit #2443 has reverted the code to a small inconsistency (as discussed in #2248).Downgrading
knitr
to a prior version sorts the issue, though...I hope this is helpful?
By filing an issue to this repo, I promise that
xfun::session_info('knitr')
. I have upgraded all my packages to their latest versions (e.g., R, RStudio, and R packages), and also tried the development version:remotes::install_github('yihui/knitr')
.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: