search

quarto-dev / quarto-cli

Open-source scientific and technical publishing system built on Pandoc.
https://quarto.org
Other
3.59k stars 294 forks source link

pagedjs-cli not working as pdf-engine #8946

Closed tvroylandt closed 4 months ago

tvroylandt commented 4 months ago

Bug description

Despite being listed as a possible PDF engine for Pandoc, pagedjs-cli doesn't work anymore as a pdf-engine since this commit : https://github.com/quarto-dev/quarto-cli/commit/89c8120cb7ba9b50ddd8d815839557014c5db299

Just adding it to the list seems to work (see https://github.com/quarto-dev/quarto-cli/commit/1be2e2f7986617ac2c832c2a9daba59671cbff57 )

Steps to reproduce

---
title: "Title"

format: 
  pdf:
    pdf-engine: pagedjs-cli
---

Some text

Expected behavior

This should works and produce a PDF

Actual behavior

It throws an error :

ERROR: Validation of YAML front matter failed.
ERROR: In file template.qmd
(line 6, columns 17--27) Field "pdf-engine" has value pagedjs-cli, which must instead be one of: `pdflatex`, `lualatex`, `xelatex`, `latexmk`, `tectonic`, `wkhtmltopdf`, `weasyprint`, `prince`, `context`, `pdfroff`
5:   pdf:
6:     pdf-engine: pagedjs-cli
                  ~~~~~~~~~~~
7: ---

ERROR: Render failed due to invalid YAML.

Your environment

Quarto check output

Quarto 1.4.550
[✓] Checking versions of quarto binary dependencies...
      Pandoc version 3.1.11: OK
      Dart Sass version 1.69.5: OK
      Deno version 1.37.2: OK
[✓] Checking versions of quarto dependencies......OK
[✓] Checking Quarto installation......OK
      Version: 1.4.550
      Path: /Applications/quarto/bin

[✓] Checking tools....................OK
      TinyTeX: (not installed)
      Chromium: (not installed)

[✓] Checking LaTeX....................OK
      Tex:  (not detected)

[✓] Checking basic markdown render....OK

[✓] Checking Python 3 installation....OK
      Version: 3.11.5
      Path: /opt/homebrew/opt/python@3.11/bin/python3.11
      Jupyter: 5.7.1
      Kernels: python3

[✓] Checking Jupyter engine render....OK

[✓] Checking R installation...........OK
      Version: 4.3.1
      Path: /opt/homebrew/Cellar/r/4.3.1/lib/R
      LibPaths:
        - /opt/homebrew/lib/R/4.3/site-library
        - /opt/homebrew/Cellar/r/4.3.1/lib/R/library
      knitr: 1.45
      rmarkdown: 2.25

[✓] Checking Knitr engine render......OK
cderv commented 4 months ago

Thanks for the report. Indeed the value is missing in the YAML schema used for validation.

Thanks for the report.

cderv commented 4 months ago

Done in b0a4230