Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 8, 2026

Description

Corrected path pattern in GitHub Actions workflow to properly ignore .github folder changes. The pattern 'github/**' does not match hidden folders - changed to '.github/**' with leading dot.

Changes:

  • Updated paths-ignore in both push and pull_request triggers
  • Lines 7 and 14 in .github/workflows/tests.yml

Without the leading dot, changes to workflow files would incorrectly trigger test runs.

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

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: stardriftfx <56492920+stardriftfx@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on Mermaid diagrams for GitHub Pages Fix path pattern to correctly ignore .github folder in workflow Jan 8, 2026
Copilot AI requested a review from stardriftfx January 8, 2026 08:34
@stardriftfx stardriftfx marked this pull request as ready for review January 8, 2026 08:34
Copilot AI review requested due to automatic review settings January 8, 2026 08:34
@stardriftfx stardriftfx merged commit 95dd621 into 893-mermaid-diagrams-gh-pages Jan 8, 2026
@stardriftfx stardriftfx deleted the copilot/sub-pr-895-again branch January 8, 2026 08:34
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 a bug in the GitHub Actions workflow configuration where the paths-ignore pattern was incorrectly specified as 'github/**' instead of '.github/**', preventing proper exclusion of workflow file changes from triggering test runs.

Key Changes:

  • Corrected the paths-ignore pattern to include the leading dot in .github/** for both push and pull_request triggers

💡 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