Skip to content

Conversation

@cscheid
Copy link
Collaborator

@cscheid cscheid commented Oct 8, 2025

(What the title says)

@github-actions github-actions bot temporarily deployed to pull request October 8, 2025 18:38 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

📝 Preview Deployment

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

🔄 Modified Documents

@github-actions github-actions bot temporarily deployed to pull request October 8, 2025 18:52 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

📝 Preview Deployment

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

🔄 Modified Documents

Copy link
Collaborator

@cwickham cwickham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

| `QUARTO_EXECUTE_INFO` | `QUARTO_EXECUTE_INFO` holds the name of a file containing a JSON object with execution information for Quarto engines. This is accessible by code in executable code cells. |
+--------------------------------------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+

For more information on the JSON object referenced by `QUARTO_EXECUTE_INFO`, see [Quarto engine execution information](./quarto-execute-info.qmd).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason this link isn't inside the table?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wanted to highlight the new page on a para by itself, but yeah, we should have the link in both places.

| `render` | object | Rendering options including `keep-tex`, `code-fold`, `fig-align`, output extensions, etc. |
| `pandoc` | object | Pandoc-specific settings like `standalone`, `to`, and `default-image-extension` |
| `language` | object | Localized text for UI elements, section titles, callouts, and other interface strings |
| `metadata` | object | Document metadata including title and format-specific options |
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"...format-specific options" is a bit confusing here because I think about it meaning options I set under a format, but here I think it means options specific to a format that aren't otherwise in one of the previous categories.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about we just cut that to "document metadata"?

| `to` | string | Pandoc output format |
| `standalone` | boolean | Whether to produce standalone output |
| `default-image-extension` | string | Default file extension for images |

Copy link
Collaborator

@cwickham cwickham Oct 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No subsection for language or metadata? It think it's worth mentioning at least custom metadata end up in metadata. Maybe something about where project-level or _metadata.yml items end up?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That information isn't in QUARTO_EXECUTE_INFO...

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean, you just said in the table format has elements metadata and language, and they don't get sections like identifier, execute, ... etc.

And if you set custom metadata like:

custom-value: "my-value"

It is in QUARTO_EXECUTE_INFO:

info_file <- Sys.getenv("QUARTO_EXECUTE_INFO")
info <- fromJSON(info_file)
info$format$metadata$`custom-value`

@github-actions github-actions bot temporarily deployed to pull request October 8, 2025 22:40 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

📝 Preview Deployment

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

🔄 Modified Documents

@github-actions github-actions bot temporarily deployed to pull request October 8, 2025 22:52 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

📝 Preview Deployment

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

🔄 Modified Documents

@github-actions github-actions bot temporarily deployed to pull request October 9, 2025 16:11 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Oct 9, 2025

📝 Preview Deployment

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

🔄 Modified Documents

@cscheid cscheid merged commit 1d33734 into prerelease Oct 9, 2025
4 checks passed
@cscheid cscheid deleted the 1.8/quarto_execute_info branch October 9, 2025 19:58
cwickham added a commit that referenced this pull request Oct 10, 2025
* QUARTO_EXECUTE_INFO documentation

* remove some lines

* Update docs/advanced/quarto-execute-info.qmd

Co-authored-by: Charlotte Wickham <[email protected]>

* Update docs/advanced/environment-vars.qmd

Co-authored-by: Charlotte Wickham <[email protected]>

* Update docs/advanced/quarto-execute-info.qmd

Co-authored-by: Charlotte Wickham <[email protected]>

* change to base_format, use fallthrough

* add link to table content

---------

Co-authored-by: Charlotte Wickham <[email protected]>
cwickham added a commit that referenced this pull request Oct 10, 2025
* QUARTO_EXECUTE_INFO documentation

* remove some lines

* Update docs/advanced/quarto-execute-info.qmd

Co-authored-by: Charlotte Wickham <[email protected]>

* Update docs/advanced/environment-vars.qmd

Co-authored-by: Charlotte Wickham <[email protected]>

* Update docs/advanced/quarto-execute-info.qmd

Co-authored-by: Charlotte Wickham <[email protected]>

* change to base_format, use fallthrough

* add link to table content

---------

Co-authored-by: Charlotte Wickham <[email protected]>
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.

3 participants