ci: scope pull request checks to relevant paths#839
Open
manuzhang wants to merge 2 commits into
Open
Conversation
manuzhang
marked this pull request as ready for review
July 22, 2026 09:14
Filter pull request checks without duplicating path lists on push triggers. Run Actions validators for GitHub YAML changes, and skip build, test, and lint workflows for documentation and development-only paths. Keep license and pre-commit checks unfiltered. Co-authored-by: Codex <codex@openai.com>
manuzhang
force-pushed
the
agent/add-workflow-path-filters
branch
from
July 22, 2026 09:18
29d8f70 to
626a80f
Compare
manuzhang
changed the title
There was a problem hiding this comment.
Pull request overview
Note
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
This PR narrows CI workflow execution using GitHub Actions path filters so that expensive checks only run when relevant files change (especially docs/dev-only vs CI config changes).
Changes:
- Scoped ASF allowlist, Actions CodeQL, and zizmor workflows to run on PRs only when
.github/**/*.yml(yaml)changes. - Added
paths-ignorefilters to multiple test/lint workflows to skip PR runs for markdown/docs/devcontainer/dev-only changes. - Expanded
cpp-linterPRpaths-ignorefilters and removedpaths-ignorefor thepushtrigger.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/zizmor.yml | Adds PR path filters so zizmor runs only for .github YAML changes. |
| .github/workflows/test.yml | Adds PR paths-ignore to skip general tests for docs/dev-only changes. |
| .github/workflows/sql_catalog_test.yml | Adds PR paths-ignore to skip SQL catalog tests for docs/dev-only changes. |
| .github/workflows/sanitizer_test.yml | Adds PR paths-ignore to skip sanitizer tests for docs/dev-only changes. |
| .github/workflows/cpp-linter.yml | Expands PR ignore patterns and removes push ignore patterns. |
| .github/workflows/codeql.yml | Adds PR path filters so CodeQL runs only for .github YAML changes. |
| .github/workflows/aws_test.yml | Adds PR paths-ignore to skip AWS tests for docs/dev-only changes. |
| .github/workflows/asf-allowlist-check.yml | Adds PR path filters so allowlist check runs only for .github YAML changes. |
Comment on lines
21
to
24
| @@ -22,17 +22,27 @@ on: | |||
| branches: | |||
| - main | |||
| paths-ignore: | |||
Comment on lines
25
to
30
| - main | ||
| paths: | ||
| - '.github/**/*.yml' | ||
| - '.github/**/*.yaml' | ||
| pull_request: | ||
| types: [opened, synchronize, reopened, ready_for_review] |
Comment on lines
28
to
33
| - '*.md' | ||
| - '**/*.md' | ||
| - 'dev/**' | ||
| - 'mkdocs/**' | ||
| pull_request: | ||
| types: [opened, synchronize, reopened, ready_for_review] |
Comment on lines
+25
to
+27
| - '**' | ||
| paths-ignore: | ||
| - '.devcontainer/**' |
Comment on lines
+28
to
+33
| paths-ignore: | ||
| - '.devcontainer/**' | ||
| - '*.md' | ||
| - '**/*.md' | ||
| - 'dev/**' | ||
| - 'mkdocs/**' |
Comment on lines
+28
to
+33
| paths-ignore: | ||
| - '.devcontainer/**' | ||
| - '*.md' | ||
| - '**/*.md' | ||
| - 'dev/**' | ||
| - 'mkdocs/**' |
Co-authored-by: Codex <codex@openai.com>
Comment on lines
+28
to
+32
| paths-ignore: | ||
| - '.devcontainer/**' | ||
| - '**/*.md' | ||
| - 'dev/**' | ||
| - 'mkdocs/**' |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
.github.Why
These workflows currently run on pull requests even when every changed path is unrelated to what they validate. Directory and extension globs keep the filters applicable to future files without maintaining filename lists, and keeping filters off push triggers avoids duplicate maintenance.
Impact
Documentation and development-only pull requests avoid unrelated CI work. Mixed changes still run the relevant workflows, GitHub YAML changes still run the Actions validators, and main, tag, and scheduled runs remain unfiltered.