Skip to content

Remove paths-ignore from pr.yaml#68

Merged
Chris-Wolfgang merged 2 commits into
mainfrom
fix/remove-md-paths-ignore
Apr 13, 2026
Merged

Remove paths-ignore from pr.yaml#68
Chris-Wolfgang merged 2 commits into
mainfrom
fix/remove-md-paths-ignore

Conversation

@Chris-Wolfgang
Copy link
Copy Markdown
Owner

Summary

  • Removes paths-ignore from the pull_request trigger in pr.yaml
  • Markdown-only PRs (e.g. CODE_OF_CONDUCT.md changes) were permanently blocked because required stage checks never triggered
  • Also removes the .github/workflows/** exclusion which would have caused the same problem for workflow-only PRs
  • Matches the repo-template standard which has no paths-ignore

Test plan

  • Verify existing open PRs get CI checks after this merges

🤖 Generated with Claude Code

Markdown-only PRs (e.g. CODE_OF_CONDUCT.md) were permanently blocked
because the required stage checks never ran. Also removes the
.github/workflows/** exclusion. Matches repo-template.
Copilot AI review requested due to automatic review settings April 13, 2026 02:47
Copy link
Copy Markdown
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

Removes paths-ignore from the pull_request trigger in the PR validation workflow so CI runs for documentation-only and workflow-only pull requests, preventing required checks from being skipped.

Changes:

  • Removed paths-ignore entries for **.md, docs/**, and .github/workflows/** from .github/workflows/pr.yaml.
  • Ensured PR CI triggers consistently align with the repo-template standard (no path-based exclusions).

@Chris-Wolfgang Chris-Wolfgang merged commit ace3150 into main Apr 13, 2026
6 checks passed
@Chris-Wolfgang Chris-Wolfgang deleted the fix/remove-md-paths-ignore branch April 13, 2026 18:51
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