-
Notifications
You must be signed in to change notification settings - Fork 352
Preview not updated on changes to included files #2795
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
Comments
It's certainly a bug, but a fix would be spread all over the quarto codebase as is. We're planning improvements for this that will make the fix easier. (in other words, it's on our radar, but don't expect it in the next 2-5 months) |
When files are included additionally as resources in |
Hi! I'm also interested in this feature. How do you declare the included files in Best. |
@cscheid can you give a (new) estimate when / if this will be fixed? |
It will be fixed when it will be fixed. The team is working on many things and time is not unlimited. |
I think a simple fix would be to do what e.g. other SSG's do and just give the user the option to include what to watch. Something like: preview:
watch:
- "pages/*.qmd"
- "resources/*.css" ...or similar. That way, this wouldn't cause a massive change / disrupt in how includes work, and let's the user decide when a re-render needs to be triggers. |
We've accumulated a fair bit of Up until recently, Quarto didn't have the knowledge about the include dependency graph, and now we do via the information in (to be clear, 1.7 and 1.8 are only expected to come out in Feb/Apr 2025.) |
@cscheid I just stumbled on this also, it really hurts the workflow you'd want. Did this make the |
Let's try to solve this for 1.8, among other issues with |
Bug description
Not sure if this qualifies as bug or feature request: running
quarto preview
on a file such asthe preview is updated when this file is changed, but not when the included
b.qmd
is changed.Tested with quarto 1.1.251 and 1.2.203 on Pop!_OS 22.04.
Checklist
The text was updated successfully, but these errors were encountered: