Skip to content

docs: clarify vocabulary page generation pipeline#136

Merged
rdhyee merged 1 commit intoisamplesorg:mainfrom
rdhyee:docs/vocab-generation-pipeline
Apr 24, 2026
Merged

docs: clarify vocabulary page generation pipeline#136
rdhyee merged 1 commit intoisamplesorg:mainfrom
rdhyee:docs/vocab-generation-pipeline

Conversation

@rdhyee
Copy link
Copy Markdown
Contributor

@rdhyee rdhyee commented Apr 24, 2026

Summary

Rewrites the Development section of the README to accurately describe the two-stage vocabulary page build:

  1. TTL → markdown via the vocab CLI from isamplesorg/vocab_tools (installed via pipx in CI), driven by scripts/generate_vocab_docs.sh.
  2. markdown → HTML via quarto render applying _quarto.yml's theme, navigation, and sidebar.

Also notes that scripts/vocab2md.py is no longer part of the build — PR #48 switched to the vocab markdown CLI entry point (same tool, same transform). The existing README paragraph still pointed at vocab2md.py, which caused confusion about which code produces the rendered vocabulary pages.

No functional changes; documentation only.

Test plan

  • Read the rendered README on GitHub and verify links resolve
  • Confirm the described pipeline matches .github/workflows/quarto-pages.yml and scripts/generate_vocab_docs.sh

🤖 Generated with Claude Code

Rewrites the Development section's vocab paragraph to describe the
actual two-stage build:

1. TTL → markdown via the `vocab` CLI from isamplesorg/vocab_tools
   (installed via pipx in CI), driven by scripts/generate_vocab_docs.sh
2. markdown → HTML via `quarto render` applying _quarto.yml's theme,
   nav, and sidebar

Also notes that scripts/vocab2md.py is no longer invoked (PR isamplesorg#48
switched to the `vocab markdown` CLI entry point — same tool, same
transform) to prevent future confusion about which code produces
the rendered vocabulary pages.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rdhyee rdhyee merged commit a4ed64b into isamplesorg:main Apr 24, 2026
1 check 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.

1 participant