Skip to content

CI: skip build/test workflows on docs-only pull requests - #8167

Open
mohanchen wants to merge 1 commit into
deepmodeling:developfrom
mohanchen:ci-skip-docs-builds
Open

mohanchen wants to merge 1 commit into
deepmodeling:developfrom
mohanchen:ci-skip-docs-builds

Conversation

@mohanchen

Copy link
Copy Markdown
Collaborator

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:

  • 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.

Reminder

  • I have read AGENTS.md and docs/developers_guide/agent_governance.md.
  • I have linked an issue or explained why this PR does not need one.
  • I have added adequate unit tests and/or case tests, or explained why not.
  • I have listed the exact verification commands run and their results.
  • I have described user-visible behavior changes, including INPUT parameter changes.
  • I have explained core-module impact for ESolver, HSolver, ElecState, Hamilt, Operator, Psi, or other source/ changes.
  • I have requested any needed governance exception below.

Linked Issue

Fix #

Unit Tests and/or Case Tests for my changes

  • Commands run:
  • Result summary:
  • Checks not run, with reason:

What's changed?

  • Example: brief summary of the user-visible or developer-facing change.

Governance Notes

  • INPUT/docs changes:
  • Core module impact:
  • Exceptions requested:

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>
@mohanchen mohanchen added the Standards Coding standards and governance rules label Oct 17, 2026
@mohanchen
mohanchen requested a review from Critsium-xy October 17, 2026 13:10
@mohanchen mohanchen added Compile & CICD & Docs & Dependencies Issues related to compiling ABACUS Tests/Examples Issues/PR related to unit tests and integrate tests labels Oct 17, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Compile & CICD & Docs & Dependencies Issues related to compiling ABACUS Standards Coding standards and governance rules Tests/Examples Issues/PR related to unit tests and integrate tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant