Skip to content

ci: run test/coverage/format workflows only on code changes - #72

Merged
MRyutaro merged 1 commit into
mainfrom
ci/test-on-code-change
Jun 17, 2026
Merged

MRyutaro merged 1 commit into
mainfrom
ci/test-on-code-change

Conversation

@MRyutaro

Copy link
Copy Markdown
Owner

概要

テスト系の CI ワークフローを「適切なタイミングでのみ」走らせるよう、paths フィルタを追加する。コード・テスト・依存・ワークフロー自体が変わったときだけ実行し、README・docs・CLAUDE.md などドキュメントのみの変更ではスキップする。push(main) / PR(main) の両イベントは維持。

背景

現状 test.yml(5バージョン matrix の pytest)・library-coverage.yml・format-python-code.yml は main への push と PR の両方で無条件に発火していた。そのため CLAUDE.md や README だけの変更でも 5 バージョンのフルテストが走り、無駄が大きかった。

変更

ワークフロー 発火する paths
test.yml elastic_kernel/** elastic_notebook/** tests/** pyproject.toml uv.lock + 自ファイル
library-coverage.yml 上記 + scripts/library_coverage.py(workflow_dispatch は維持)
format-python-code.yml **/*.py pyproject.toml + 自ファイル

安全性

  • main に required status check は設定されていない(確認済み)ため、docs のみ変更の PR でテストがスキップされてもマージはブロックされない。
  • ワークフロー自体のパスを含めているので、トリガー設定を今後変更したときはそのコミットで再実行される。
  • 3ファイルとも YAML パース検証済み。

🤖 Generated with Claude Code

push/PR(main) のテスト系ワークフローに paths フィルタを追加し、コード・テスト・依存・
ワークフロー自体が変わったときだけ走らせる。README/docs/CLAUDE.md などドキュメントのみの
変更ではスキップする(5バージョン matrix の無駄打ちを抑制)。

- test.yml: elastic_kernel/ elastic_notebook/ tests/ pyproject.toml uv.lock + 自ファイル
- library-coverage.yml: 上記 + scripts/library_coverage.py(workflow_dispatch は維持)
- format-python-code.yml: **/*.py pyproject.toml + 自ファイル

main に required status check は設定されていないため、docs のみ変更の PR が
スキップされてもマージはブロックされない。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@MRyutaro
MRyutaro merged commit d0f9249 into main Jun 17, 2026
7 checks passed
@MRyutaro
MRyutaro deleted the ci/test-on-code-change branch June 17, 2026 07:34
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