Skip to content

[Sync to prerelease] Treat any non-markdown engine as computational in freeze guidance - #2129

Merged
cderv merged 1 commit into
prereleasefrom
sync-2128-to-prerelease
Jul 24, 2026
Merged

[Sync to prerelease] Treat any non-markdown engine as computational in freeze guidance#2129
cderv merged 1 commit into
prereleasefrom
sync-2128-to-prerelease

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Description

Sync of #2128 to prerelease.

Original PR

Treat any non-markdown engine as computational in freeze guidance

The _freeze/ guidance in .claude/rules/quarto-web-workflow.md named knitr/jupyter/julia as the executing engines. But since Quarto 1.9's Engine Extensions a document can use a custom engine beyond those, and markdown is the only non-executing engine (the no-op default when nothing else claims the file).

With the old allow-list phrasing, a custom-engine page would be misread as non-computational — and its real freeze could be deleted as "vestigial".

Reframe as an invariant: quarto inspect's engines array is markdown ⇒ no execution; any other engine ⇒ the page executes code (with knitr/jupyter/julia as examples, not an exhaustive list).

Follows #2124.

)

The _freeze guidance named knitr/jupyter/julia as the executing engines, but since Quarto 1.9 Engine Extensions a page can use a custom engine beyond those. markdown is the only non-executing engine (the no-op default), so a custom-engine page would be misread as non-computational and risk deleting a real freeze. Reframe as an invariant: markdown means no execution, any other engine executes.

(cherry picked from commit 21ec13a)
@cderv

cderv commented Jul 24, 2026

Copy link
Copy Markdown
Member

/deploy-preview

@cderv
cderv merged commit 8a07f5c into prerelease Jul 24, 2026
@cderv
cderv deleted the sync-2128-to-prerelease branch July 24, 2026 21:19
@github-actions

Copy link
Copy Markdown
Contributor Author

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-2129.quarto.org

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant