Set CI=true for Quarto TinyTeX install command#627
Merged
Conversation
Quarto renders an animated TinyTeX download progress bar unless its `runningInCI()` check passes. That check only reads environment variables (CI, GITHUB_ACTIONS, etc.), none of which propagate into a `docker build` RUN step, so inside container builds Quarto thinks it is interactive and redraws the progress bar on every download tick. Written to a non-TTY build log, each redraw becomes a separate line — a single observed build emitted ~26k of them, ~5MB of redundant output that slows the build and bloats CI logs. Set `CI=true` inline on the install command so `runningInCI()` passes and Quarto collapses the progress bar to one line. Unlike re-adding `--quiet` (removed so install errors stay visible), this keeps all error output. The variable is scoped to the command and does not leak into the built image.
Test Results1 896 tests 1 896 ✅ 11m 22s ⏱️ Results for commit 546ca0d. |
Contributor
Author
|
Merging this to fix long-running workbench dev version builds Will have follow-up PRs in the individual repos. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Quarto renders an animated TinyTeX download progress bar unless its
runningInCI()check passes. That check only reads environment variables (CI,GITHUB_ACTIONS, etc.), none of which propagate into adocker buildRUN step — so inside container builds Quarto thinks it is interactive and redraws the progress bar on every download tick.In a non-TTY build log each redraw becomes its own line. One observed Workbench dev build emitted ~26,000 of them (~5 MB, 68% of the log), which slows the build and bloats CI logs.
Setting
CI=trueinline on the install command makesrunningInCI()pass, so Quarto collapses the progress bar to a single line. Unlike re-adding--quiet(removed in 51c7fc6 so install errors stay visible), this keeps all error output, and the variable is scoped to the command so it does not leak into the built image.