Skip to content

Fix CI running on docs-only PRs#14

Merged
rickcrawford merged 1 commit intomainfrom
fix/ci-skip-docs-only
Apr 12, 2026
Merged

Fix CI running on docs-only PRs#14
rickcrawford merged 1 commit intomainfrom
fix/ci-skip-docs-only

Conversation

@rickcrawford
Copy link
Copy Markdown
Contributor

Summary

  • Replace paths-ignore at workflow level with dorny/paths-filter at job level
  • When paths-ignore skips a workflow entirely, GitHub never reports a status, so required checks like build stay stuck at "Waiting for status to be reported" forever
  • With job-level filtering, the workflow always runs and reports a skipped status, which satisfies branch protection

Fixes the issue seen in #13 where a README-only PR can't be merged.

Files changed

  • .github/workflows/ci.yml - add changes job, conditional build job
  • .github/workflows/codeql.yml - add changes job, conditional analyze job

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

Use dorny/paths-filter at job level instead of paths-ignore at workflow
level. When paths-ignore skips a workflow entirely, GitHub never reports
a status, so required checks like "build" stay stuck at "Waiting"
forever. With job-level filtering, the workflow always runs and reports
a skipped status, which satisfies branch protection.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@rickcrawford rickcrawford merged commit c7833e9 into main Apr 12, 2026
8 checks passed
@rickcrawford rickcrawford deleted the fix/ci-skip-docs-only branch April 12, 2026 23:02
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.

1 participant