Skip to content

ci: scope composite-action path filters to their suite#310

Merged
pratyush618 merged 1 commit into
masterfrom
ci/narrow-path-filters
Jun 27, 2026
Merged

ci: scope composite-action path filters to their suite#310
pratyush618 merged 1 commit into
masterfrom
ci/narrow-path-filters

Conversation

@pratyush618

@pratyush618 pratyush618 commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

What

Narrow the changes job's path filters so a composite-action edit only reruns the suites that actually use it.

Why

shared matched .github/actions/**, and every suite includes *shared — so editing any composite action (e.g. setup-java) reran rust + python + node + java. The setup-java@v5 bump in #309 fired the whole Python/Node matrix for nothing.

Research consensus: over-triggering wastes minutes, under-triggering ships untested bugs. Narrowing is safe only for single-consumer actions; the map was grep-verified:

action consumers
setup-rust rust, python, node, java → stays in shared
setup-python rust, python
rust-prelude rust
setup-node node
setup-java java

Change

shared keeps only setup-rust; each suite lists the actions it consumes. No suite loses a dependency it had (no under-trigger), single-consumer actions stop fanning out to all four (no over-trigger).

Summary by CodeRabbit

  • Chores
    • Updated CI change-detection so jobs run only for the suites and workflow files they actually relate to.
    • Reduced unnecessary CI reruns by narrowing shared triggers and adding suite-specific workflow/action paths.

@github-actions github-actions Bot added the ci label Jun 26, 2026
@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 78979d9b-eed3-439e-a9dd-2c7cc8b0a197

📥 Commits

Reviewing files that changed from the base of the PR and between 8f06d7b and 4eeaef2.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

The CI workflow now uses narrower path filters so suites rerun only for their own code, composite action, and workflow file changes. The shared filter keeps Rust crates, root manifests, setup-rust, and the CI workflow file.

Changes

CI path filter changes

Layer / File(s) Summary
Narrowed suite filters
.github/workflows/ci.yml
The dorny/paths-filter rules replace the broad .github/actions/** trigger with shared and suite-scoped action and workflow paths.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • ByteVeda/taskito#304 — Also changes the CI workflow path-filtering logic that decides when suite jobs rerun.

Poem

A bunny hops through CI light,
🐇 filters neat, all set just right.
Rust and suites now wake when near,
not every action loud and clear.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: narrowing CI composite-action path filters to the suites that use them.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/narrow-path-filters

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

@pratyush618
pratyush618 merged commit 7551eaa into master Jun 27, 2026
31 checks passed
@pratyush618
pratyush618 deleted the ci/narrow-path-filters branch June 27, 2026 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant