Skip to content

ci: paths-filter で root 直下の .md も docs 扱いにする(#59)#60

Merged
yk-lab merged 1 commit into
mainfrom
ci/paths-filter-root-md
Jun 28, 2026
Merged

ci: paths-filter で root 直下の .md も docs 扱いにする(#59)#60
yk-lab merged 1 commit into
mainfrom
ci/paths-filter-root-md

Conversation

@yk-lab

@yk-lab yk-lab commented Jun 28, 2026

Copy link
Copy Markdown
Owner

概要

ci.yml / self-test.ymldorny/paths-filter フィルタ !**/*.mdroot 直下の .mdCLAUDE.md / TODO.md / README.md 等)にマッチしないため、これらだけの PR でも src = true と判定され、フル matrix / self-test / proxy ジョブが走って CI 課金を無駄にしていた。

実例: PR #58CLAUDE.md + TODO.md のみ)の Detect changesFilter src = true

変更内容

  • ci.yml / self-test.ymlchanges フィルタに - '!*.md' を追加(root 直下の .md を除外)。!**/*.md(サブディレクトリの .md)と併用

検証

  • actionlint ✅ / typos
  • 本 PR は .yml を変更するため src = true でフル CI が走る(=挙動は次の docs-only PR で「重いジョブがスキップされる」ことで確認できる)
  • 非 .md を含む PR は従来どおりフル CI(回帰なし)

参照

🤖 Generated with Claude Code


Summary by cubic

Exclude root-level .md files from the dorny/paths-filter src filter so docs-only PRs skip heavy CI (matrix/self-test/proxy); non-doc PRs are unchanged. Implemented by adding !*.md alongside !**/*.md in ci.yml and self-test.yml. Closes #59.

Written for commit 9912931. Summary will update on new commits.

Review in cubic

`!**/*.md` は root 直下の .md(CLAUDE.md / TODO.md / README.md 等)にマッチ
しないため、それらだけの PR でも src=true 判定でフル CI が走っていた
(PR #58 で確認)。`!*.md` を追加して root の .md も除外する。

- ci.yml / self-test.yml の changes フィルタに `!*.md` を追記
- TODO.md: #59 を解決済み(#60)へ

Closes #59

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AECzAv2vcdATecuvLjBSa6
@coderabbitai

coderabbitai Bot commented Jun 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 362b9089-bf9d-4a44-9897-83da18674b90

📥 Commits

Reviewing files that changed from the base of the PR and between ecb3f23 and 9912931.

⛔ Files ignored due to path filters (2)
  • .github/workflows/ci.yml is excluded by !**/*.yml
  • .github/workflows/self-test.yml is excluded by !**/*.yml
📒 Files selected for processing (1)
  • TODO.md

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • 既に解決済みの Issue 一覧に、[ci]paths-filter に関する項目が1件追加されました。
    • 既存の記載内容に変更はありません。

Walkthrough

TODO.md の「既に解決した Issue」テーブルに、paths-filter が root 直下の .md ファイルを docs 扱いせずフル CI を走らせる問題(Issue #59)に関する行が1行追加された。

Changes

TODO.md 更新

Layer / File(s) Summary
解決済み Issue #59 の記録追加
TODO.md
paths-filter が root 直下の .md を docs 扱いしない問題(#59)を「既に解決した Issue」テーブルに追記。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minutes

Poem

🐰 ふわふわドキュメント、一行追加
TODO テーブルに刻まれた記録
paths-filter の罠、今は昔
うさぎはぴょんと解決済みへ
✅ 平和な CI の日々が続く

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Linked Issues check ❓ Inconclusive 必要な .github/workflows/ci.yml と self-test.yml が path フィルタでレビュー対象外のため、要件達成を確認できない。 対象ワークフローの変更を別途確認できる差分か、レビュー対象に含めてください。
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 変更の中心である paths-filter の Markdown 判定修正を簡潔に表している。
Description check ✅ Passed 概要、関連 issue、検証、参照が揃っており、テンプレート要件を概ね満たしている。
Out of Scope Changes check ✅ Passed 確認できる変更は TODO.md の issue 追記のみで、目的に沿った範囲内に収まっている。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/paths-filter-root-md

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Re-trigger cubic

@yk-lab yk-lab merged commit 33f0ea3 into main Jun 28, 2026
21 checks passed
@yk-lab yk-lab deleted the ci/paths-filter-root-md branch June 28, 2026 02:59
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.

[ci] paths-filter が root 直下の .md を docs 扱いせずフル CI が走る

1 participant