Repository navigation
ci: limit build matrix to relevant paths - #19
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 38cccf77e6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - '.github/workflows/ci.yml' | ||
| pull_request: | ||
| branches: [ main, master ] | ||
| paths: |
There was a problem hiding this comment.
Keep required checks present for filtered pull requests
When a matrix job from this workflow is configured as a required branch-protection check, a pull request that changes only an excluded path (such as docs/**) skips the entire workflow, so GitHub leaves the expected check pending and the pull request cannot be merged. Keep the pull_request trigger unconditional and perform path detection within the workflow, with an always-reported gate job, so documentation-only changes can satisfy required checks without running the build matrix.
Useful? React with 👍 / 👎.
Summary
Recreates the CI path-filter change cleanly on top of current
mainafter PR #17 was merged and the original stacked base branch was deleted.CI now runs for
include/**src/**tests/**benchmarks/**examples/**cmake/**CMakeLists.txtvcpkg.jsonvcpkg-configuration.json.github/workflows/ci.ymlThe same allowlist applies to pushes to
main/masterand pull requests targeting those branches.CI no longer runs for documentation-only changes
Examples:
docs/**README.mdRecovery note
The original PR #18 was stacked on top of
refactor/src-layout. After #17 merged, that temporary base branch was deleted, so GitHub could not reopen/retarget #18. This PR is recreated from currentmainand contains only.github/workflows/ci.yml.