Skip to content

Commit 4aeb8c2

Browse files
ci(pr): exclude skills/**/evals/** from test-trigger filters (#1170)
## Summary The `changed-files` filter groups in `.github/workflows/pr.yaml` (`test_cpp`, `test_python_conda`, `test_python_wheels`) already exclude `skills/**/SKILL.md` and `skills/**/resources/**` so doc-only changes to those paths skip the build/test pipeline. The same exclusion was missing for `skills/**/evals/**`, so adding or editing an `evals.json` unnecessarily triggers `conda-cpp-build`, `conda-python-build`, the wheel builds, and downstream tests. This PR adds `'!skills/**/evals/**'` to the three test-trigger filter groups, matching the treatment of other non-buildable skill content. ## Notes - `build_docs` is intentionally **not** changed — it does not exclude `SKILL.md` or `resources/**` either, so docs-build behavior is unchanged for all skill content. - Verified by: - `python3 -c "import yaml; yaml.safe_load(open('.github/workflows/pr.yaml'))"` — YAML parses - `pre-commit run --files .github/workflows/pr.yaml` — all hooks pass Signed-off-by: Ramakrishna Prabhu <[email protected]>
1 parent 746f46b commit 4aeb8c2

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

.github/workflows/pr.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -223,6 +223,7 @@ jobs:
223223
- '!notebooks/**'
224224
- '!python/**'
225225
- '!skills/**/SKILL.md'
226+
- '!skills/**/evals/**'
226227
- '!skills/**/resources/**'
227228
- '!sonar-project.properties'
228229
- '!sonarqube/**'
@@ -282,6 +283,7 @@ jobs:
282283
- '!ngc/**'
283284
- '!notebooks/**'
284285
- '!skills/**/SKILL.md'
286+
- '!skills/**/evals/**'
285287
- '!skills/**/resources/**'
286288
- '!sonar-project.properties'
287289
- '!sonarqube/**'
@@ -342,6 +344,7 @@ jobs:
342344
- '!ngc/**'
343345
- '!notebooks/**'
344346
- '!skills/**/SKILL.md'
347+
- '!skills/**/evals/**'
345348
- '!skills/**/resources/**'
346349
- '!sonar-project.properties'
347350
- '!sonarqube/**'

0 commit comments

Comments
 (0)