Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 8, 2026

Description

The {% if site.mermaid %} conditional in docs/_includes/head-custom.html was missing its closing {% endif %} tag, causing invalid Liquid template syntax.

Changes:

  • Added {% endif %} at line 22 to properly close the conditional block
{% if site.mermaid %}
<script type="module">
  // ... mermaid configuration ...
</script>
{% endif %}

Checklist (Mandatory for new features)

  • Added Documentation
  • Added Unit Tests

Testing (Mandatory for all changes)

  • GPU Test: test-medium-connected.xml Passed
  • GPU Test: test-large-long.xml Passed

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: stardriftfx <56492920+stardriftfx@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Mermaid diagrams for GitHub Pages display issues Fix unclosed Liquid conditional tag in Mermaid configuration Jan 8, 2026
Copilot AI requested a review from stardriftfx January 8, 2026 08:33
@stardriftfx stardriftfx marked this pull request as ready for review January 8, 2026 08:38
Copilot AI review requested due to automatic review settings January 8, 2026 08:38
@stardriftfx stardriftfx merged commit 06e3b28 into 893-mermaid-diagrams-gh-pages Jan 8, 2026
2 checks passed
@stardriftfx stardriftfx deleted the copilot/sub-pr-895 branch January 8, 2026 08:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes a critical syntax error in the Liquid template by adding the missing closing tag for a conditional block.

Key Changes:

  • Added the missing {% endif %} tag to properly close the {% if site.mermaid %} conditional that was opened on line 1

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants