ci(pr): exclude skills/**/evals/** from test-trigger filters

[rgsl888prabhu]

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

[copy-pr-bot]

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 3cc63529-15ff-4886-840a-defc7ea1bfbd

📥 Commits

Reviewing files that changed from the base of the PR and between 746f46b and f3816ea.

📒 Files selected for processing (1)

• .github/workflows/pr.yaml

---

📝 Walkthrough

Walkthrough

The workflow configuration in .github/workflows/pr.yaml is modified to add an exclusion pattern !skills/**/evals/** to the change-group globs for build_docs, test_cpp, test_python_conda, and test_python_wheels, preventing these CI jobs from being triggered by changes solely within the evals subdirectories.

Changes

Cohort / File(s)	Summary
CI Configuration .github/workflows/pr.yaml	Added exclusion pattern !skills/**/evals/** to four change-group globs (build_docs, test_cpp, test_python_conda, test_python_wheels) to skip CI jobs when only evaluation-related files are modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly and specifically describes the main change: adding exclusion patterns for skills//evals/ to CI test-trigger filters in the workflow file.
Description check	✅ Passed	The description clearly relates to the changeset by explaining the rationale for excluding skills//evals/ from test triggers and noting which groups are affected.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Review rate limit: 9/10 reviews remaining, refill in 6 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}