Repository navigation
Conversation
Add paths-ignore to the pull_request triggers of the six heavy build and test workflows (CMake builds, Makefile build, integration/unit tests, CUDA build+test, PyTest, ASE plugin test). Pull requests that only touch docs/**, Markdown files, or .coderabbit.yaml no longer spend runner time on compilation and physics tests that cannot be affected by such changes. Motivated by docs-only PRs such as deepmodeling#8148, which triggered 14 full build/test runs. Scope notes: - push triggers (including the develop safety-net runs) are unchanged. - agent_governance.yml is deliberately untouched: AGENTS.md changes are its primary input and must keep running. - Workflow file changes themselves are not ignored, so CI changes still trigger CI. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
AsTonyshment
left a comment
There was a problem hiding this comment.
With the current branch protection, a documentation-only PR would probably skip the workflow but still require Test to pass, leaving it waiting for a status and blocking normal merging (we'll see something like "Expected / Waiting for status to be reported").
See GitHub's "Handling skipped but required checks": https://docs.github.com/en/pull-requests/how-tos/merge-and-close-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks
|
Branch protection on
A workflow skipped by Note: moving the skip to a job-level Possible fixes:
Minor: |
Add paths-ignore to the pull_request triggers of the six heavy build and test workflows (CMake builds, Makefile build, integration/unit tests, CUDA build+test, PyTest, ASE plugin test). Pull requests that only touch docs/**, Markdown files, or .coderabbit.yaml no longer spend runner time on compilation and physics tests that cannot be affected by such changes. Motivated by docs-only PRs such as #8148, which triggered 14 full build/test runs.
Scope notes:
Reminder
AGENTS.mdanddocs/developers_guide/agent_governance.md.source/changes.Linked Issue
Fix #
Unit Tests and/or Case Tests for my changes
What's changed?
Governance Notes