Skip to content

Align YAML header convention across templates, editor-manual, and DOCS/#41

Open
MatMatt wants to merge 3 commits into
developfrom
fix/yaml-header-convention
Open

Align YAML header convention across templates, editor-manual, and DOCS/#41
MatMatt wants to merge 3 commits into
developfrom
fix/yaml-header-convention

Conversation

@MatMatt

@MatMatt MatMatt commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator

Problem

The YAML header convention was inconsistent across three sources:

  • Templates (_meta/templates/) used double quotes, product-name, a format: block, placeholder version text
  • Editor-manual forbade version in YAML, required product-name, omitted category/toc/toc-depth/toc-title
  • Actual DOCS/ on main use no quotes, category first, toc group, version present

Changes

editor-manual_v1.qmd

  • YAML example updated to match actual convention
  • version moved from forbidden to required
  • category, toc, toc-depth, toc-title documented
  • product-name removed
  • Contradictory versioning paragraph fixed

Templates (PUM + ATBD)

  • Stripped double quotes, added category/toc/version, removed product-name

Small_Landscape_Features_2021_PUM_v1.qmd

  • Fixed date from DD/MM/YYYY to ISO format to pass frontmatter validation

No other DOCS/ content files touched.

MatMatt added 2 commits July 5, 2026 22:17
Resolve conflicts between three sources that defined different YAML
header formats. Standardize on the convention actually used by all 25
DOCS/ files on main: no double quotes, category first, toc group present,
version in YAML (not forbidden).

- editor-manual: update YAML example, move version from forbidden to
  required, add category/toc fields, remove product-name
- PUM/ATBD templates: strip double quotes, add missing fields, remove
  product-name, clean up placeholder text
@MatMatt
MatMatt requested a review from mckeea as a code owner July 5, 2026 20:18

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f8289bc11b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +81 to +84
toc: true
toc-depth: 3
toc-title: Content
version: 1.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Don't require fields the build strips or overwrites

In the publish path I checked deploy-docs.yml's Build Docs step: build-docs.sh runs strip_unknown_frontmatter.py, which drops toc, toc-depth, and toc-title, and then fill_version.py, whose header says the source version: is overwritten from .llm_cache/versions.json; validate_qmd_files.py also explicitly does not check these fields. Listing them as required here will send editors to maintain metadata that production ignores or replaces, so source changes like toc-title: Content or manual version edits won't affect the deployed document.

Useful? React with 👍 / 👎.

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.

1 participant