Conversation
There was a problem hiding this comment.
Pull request overview
Adds a new documentation page to help users migrate legacy .httkweb.html templates and metadata patterns to the project’s Jinja2-based template system, and links it from the main docs index and “How it works” page.
Changes:
- Introduces a new migration guide (
migration_legacy_to_jinja2.rst) mapping legacy formatter constructs to Jinja2 equivalents. - Updates
docs/index.rstto include the new guide in quick links and the documentation toctree. - Adds a cross-reference from
docs/how_it_works.rstto the new migration guide.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/migration_legacy_to_jinja2.rst | New migration guide covering template naming, variable access, escaping, loops, conditionals, helper calls, and metadata conventions. |
| docs/index.rst | Adds the migration guide to quick links and the main documentation toctree. |
| docs/how_it_works.rst | Adds a “See also” link to the migration guide for discoverability. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 11 changed files in this pull request and generated 6 comments.
Comments suppressed due to low confidence (1)
docs/reference.md:4
:orphan:at the top of a MyST Markdown file is likely rendered as literal text (andreferenceis already included in the toctree fromindex.md). Consider removing it, or if you really need an orphan marker in Markdown, use MyST/Sphinx-supported front matter instead.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 14 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…d clarify template repo docs
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
docs/reference.md:1
- The
:orphan:line is reStructuredText metadata; in a MyST/Markdown file it will render as literal text and likely isn’t doing what you intend. If the goal is to mark the page as orphaned, use MyST front-matter metadata (---\norphan: true\n---) or just remove it (this page is already included in the toctree).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.