quarto-dev / quarto-dev/quarto-cli

our YAML validator doesn't work well with YAML references

Open
#8,762 1 comment 0 reactions 1 assignee View on GitHub

@cscheid is already working on this.

Since Feb 16, 2024.

bug yaml-validation
Dominant language
JavaScript
Stars
6k
Forks
458
Avg merge
1d 9h
Merged PRs (30d)
41

Description

I'd like to do the following:

image

Notice how html and dashboard fail our validation. The frontmatter still parses and works, but our validator doesn't know that those entries aren't empty objects.

This fix is going to be tricky because YAML references need to be verified against multiple schemas, and the source code information tracking will be hard (how do we signal an error inside the reference differently from an error in the definition?). But we should try to do it next time we're improving our YAML validation.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.