Skip to content

skip pytest in ci for docs-only pull requests#136

Merged
HumanBean17 merged 2 commits into
masterfrom
feat/run-dont-run-ci-on-docs
May 15, 2026
Merged

skip pytest in ci for docs-only pull requests#136
HumanBean17 merged 2 commits into
masterfrom
feat/run-dont-run-ci-on-docs

Conversation

@HumanBean17
Copy link
Copy Markdown
Owner

Summary

  • Add dorny/paths-filter@v3 to .github/workflows/test.yml so documentation-only PRs skip pip install and pytest while the required test job still completes green.
  • Docs-only paths include propose/, plans/, reports/, .cursor/, docs/, and all **/*.md / **/*.mdc; code paths (Python, deps, workflows, non-markdown under tests/ / automation/) still run the full suite.
  • Update tests/README.md and plan-prompts / pr-review skills for the new CI behavior (UC15).

Test plan

  • Merge this PR and confirm CI runs full pytest tests (this PR touches workflow YAML → code filter matches).
  • Follow-up docs-only PR (e.g. README-only) should show green test with "Docs-only change; pytest skipped." in the log.
  • Mixed PR (e.g. propose/foo.md + server.py) should run full pytest.

Made with Cursor

HumanBean17 and others added 2 commits May 15, 2026 13:44
Use paths-filter so propose/plans/docs PRs still get a green test check
without installing deps or running the full suite; code changes unchanged.

Co-authored-by: Cursor <cursoragent@cursor.com>
Root pytest.ini, mcp.json.example, and .gitignore changes now trigger
the full suite instead of being treated as docs-only.

Co-authored-by: Cursor <cursoragent@cursor.com>
@HumanBean17 HumanBean17 merged commit 7fcf6c4 into master May 15, 2026
1 check passed
@HumanBean17 HumanBean17 deleted the feat/run-dont-run-ci-on-docs branch May 23, 2026 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant