Skip to content

Conversation

@stardriftfx
Copy link
Contributor

Closes #893

Description

Copilot AI review requested due to automatic review settings January 8, 2026 08:18
@stardriftfx stardriftfx linked an issue Jan 8, 2026 that may be closed by this pull request
@stardriftfx stardriftfx requested a review from stiber January 8, 2026 08:19
@stardriftfx stardriftfx self-assigned this Jan 8, 2026
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 PR fixes Mermaid diagram rendering on GitHub Pages and optimizes the CI workflow to skip tests for documentation-only changes.

Key Changes:

  • Updated documentation link from GitFlowDiagram.md to GitFlow.md
  • Added Mermaid.js integration via custom head include file
  • Configured GitHub Actions to ignore documentation changes when running tests

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
docs/index.md Updated GitFlow documentation link from GitFlowDiagram.md to GitFlow.md
docs/_includes/head-custom.html Added Mermaid.js script to enable diagram rendering on GitHub Pages
docs/_config.yml Enabled Mermaid support via configuration flag
.github/workflows/tests.yml Added paths-ignore filters to skip test runs for documentation changes

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

Copy link
Contributor

Copilot AI commented Jan 8, 2026

@stardriftfx I've opened a new pull request, #896, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Contributor

Copilot AI commented Jan 8, 2026

@stardriftfx I've opened a new pull request, #897, to work on those changes. Once the pull request is ready, I'll request review from you.

stardriftfx and others added 6 commits January 8, 2026 00:30
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: stardriftfx <56492920+stardriftfx@users.noreply.github.com>
Co-authored-by: stardriftfx <56492920+stardriftfx@users.noreply.github.com>
Fix path pattern to correctly ignore .github folder in workflow
Fix unclosed Liquid conditional tag in Mermaid configuration
Copy link
Contributor

@stiber stiber left a comment

Choose a reason for hiding this comment

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

This looks great. Could you add documentation for this to docs/Developer/GHPages.md? Also, given our conversation about the gh-pages branch, I just want to confirm that this will work given our orphan branch configuration for publishing...

@stardriftfx
Copy link
Contributor Author

This looks great. Could you add documentation for this to docs/Developer/GHPages.md? Also, given our conversation about the gh-pages branch, I just want to confirm that this will work given our orphan branch configuration for publishing...

Done. And yes, based on my forked repo, this should work without modifying the orphan branch's config (gh-pages).

@stardriftfx stardriftfx merged commit 6a02426 into master Jan 16, 2026
@stardriftfx stardriftfx deleted the 893-mermaid-diagrams-gh-pages branch January 16, 2026 04:12
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.

Mermaid diagrams not rendering properly

3 participants