Skip to content

Q-20-2: quote only pandoc's version line in the too-old message - #1108

Merged
cscheid merged 2 commits into
quarto-dev:mainfrom
EwoutH:bugfix/pandoc-version-message
Nov 7, 2026
Merged

cscheid merged 2 commits into
quarto-dev:mainfrom
EwoutH:bugfix/pandoc-version-message

Conversation

@EwoutH

@EwoutH EwoutH commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

The Q-20-2 "Pandoc Version Too Old" message quoted the whole pandoc --version stdout, so on Windows it read:

pandoc reported version "pandoc.exe 3.5\r\nFeatures: +server +lua\r\nScripting engine: Lua 5.4\r\n...", which is older than the minimum required version 3.12.

It now quotes only the first line: pandoc reported version "pandoc.exe 3.5", which is older than the minimum required version 3.12.

Added a unit test (fails before the fix, passes after). cargo test -p quarto-core --lib pandoc_filters::version and cargo clippy -p quarto-core --all-targets -- -D warnings pass. I didn't rebuild the q2 binary to check the message end to end.

The message quoted the whole `pandoc --version` stdout with `{:?}`, so it
showed every feature line plus escaped `\r\n` on Windows:

  pandoc reported version "pandoc.exe 3.5\r\nFeatures: +server +lua\r\n...",
  which is older than the minimum required version 3.12.

It now quotes the first line: pandoc reported version "pandoc.exe 3.5".

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@posit-snyk-bot

posit-snyk-bot commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cscheid
cscheid merged commit e95a3ef into quarto-dev:main Nov 7, 2026
15 checks passed
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