Document the Quarto version requirement for llms-txt#107
Conversation
Downstream users on older Quarto versions hit a schema error because `llms-txt` is not a recognised `website:` property before Quarto 1.9.36. Add a note in the Prerequisites section of README.Rmd (and regenerate README.md) stating the minimum verified version. Closes #101 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Claude finished @d-morrison's task in 2m 6s —— View job Code Review
Overall: Clean, minimal doc-only PR. The addition is accurate and placed correctly in the Prerequisites section. Findings
The phrase "the version this template was verified on" implies uncertainty: readers with 1.9.34 or 1.9.35 won't know whether they're safe. If 1.9.36 is genuinely the first release to include
If instead 1.9.36 is just what was tested (and the feature could exist in earlier patch releases), it's worth noting that distinction explicitly — e.g., "requires Quarto 1.X or newer; developed on 1.9.36". Everything else ✓
Since the PR is already merged, no action is required unless you want to follow up with a small prose tweak. |
There was a problem hiding this comment.
Pull request overview
Documents the Quarto version prerequisite introduced by enabling the website: llms-txt option, helping downstream template users avoid schema errors on older Quarto releases (per #101).
Changes:
- Add a Quarto version requirement note to
README.Rmddescribing thellms-txtschema error on older Quarto versions. - Regenerate
README.mdto keep it in sync withREADME.Rmd(and satisfycheck-readme).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| README.Rmd | Adds a Quarto version requirement note tied to llms-txt. |
| README.md | Regenerated output reflecting the new prerequisite note. |
| quarto --version | ||
| ``` | ||
|
|
||
| **Minimum version:** This template uses Quarto's [`llms-txt`](https://quarto.org/docs/websites/website-llms.html) website option, which requires **Quarto 1.9.36 or newer** (the version this template was verified on). Earlier Quarto versions do not recognize `llms-txt` as a valid `website:` property and will produce a schema error when rendering. |
| **Minimum version:** This template uses Quarto's | ||
| [`llms-txt`](https://quarto.org/docs/websites/website-llms.html) website | ||
| option, which requires **Quarto 1.9.36 or newer** (the version this | ||
| template was verified on). Earlier Quarto versions do not recognize | ||
| `llms-txt` as a valid `website:` property and will produce a schema | ||
| error when rendering. |
Draft PR opened by
@claudeto address #101.Triggered by workflow run.
Addresses #101.