Skip to content

Add quiet parameter to install_tinytex_command#508

Merged
bschwedler merged 1 commit into
mainfrom
tinytex-quiet-param
May 8, 2026
Merged

Add quiet parameter to install_tinytex_command#508
bschwedler merged 1 commit into
mainfrom
tinytex-quiet-param

Conversation

@bschwedler
Copy link
Copy Markdown
Contributor

Summary

  • Add optional quiet parameter to the install_tinytex_command Jinja2 macro
  • Defaults to True so existing callers are unaffected
  • Passing quiet=False omits --quiet from the quarto install tinytex command, making install errors visible in build logs

Discovered while building scratch-based volume images in posit-dev/images-volumes — TinyTeX install failures were invisible because --quiet swallowed all output including errors.

@bschwedler bschwedler requested a review from ianpittwood as a code owner April 30, 2026 19:26
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026

Test Results

1 464 tests   1 464 ✅  8m 17s ⏱️
    1 suites      0 💤
    1 files        0 ❌

Results for commit 4995d3a.

♻️ This comment has been updated with latest results.

The --quiet flag was hardcoded, making it impossible to see
TinyTeX install errors in build logs. Add an optional quiet
parameter (defaults to True) so callers can opt into verbose
output when debugging install failures.
@ianpittwood ianpittwood force-pushed the tinytex-quiet-param branch from f26e933 to 4995d3a Compare May 4, 2026 13:22
@bschwedler bschwedler added this pull request to the merge queue May 8, 2026
Merged via the queue into main with commit 7b77cb3 May 8, 2026
39 of 42 checks passed
@bschwedler bschwedler deleted the tinytex-quiet-param branch May 8, 2026 14:31
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.

2 participants