Repository navigation
ci: run test/coverage/format workflows only on code changes - #72
Merged
Merged
Conversation
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
テスト系の 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 バージョンのフルテストが走り、無駄が大きかった。変更
test.ymlelastic_kernel/**elastic_notebook/**tests/**pyproject.tomluv.lock+ 自ファイルlibrary-coverage.ymlscripts/library_coverage.py(workflow_dispatchは維持)format-python-code.yml**/*.pypyproject.toml+ 自ファイル安全性
mainに required status check は設定されていない(確認済み)ため、docs のみ変更の PR でテストがスキップされてもマージはブロックされない。🤖 Generated with Claude Code