From 24c6ee06556753e115a61bf4712ec3efba255b35 Mon Sep 17 00:00:00 2001 From: Mikkeli Date: Fri, 4 Sep 2026 19:26:35 +0900 Subject: [PATCH] ci: path-filter the pull_request trigger too MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The push trigger was already restricted to master and to source paths, but pull_request carried only a branch filter — so a PR touching nothing but the README would have run a full build. A branch filter constrains where a workflow fires, not what fires it. Both filters now list the same six paths. A docs-only change runs nothing, on either trigger. Note that .github/workflows/** is deliberately absent from both lists, so editing this file does not run it. That is the intended trade: fewer pointless builds, at the cost of a broken workflow surfacing on the next source change rather than immediately. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_01WHt4DykxXaQnyaJfyS8KCn --- .github/workflows/ci.yaml | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 51fb9e2..dbc92f4 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -10,8 +10,17 @@ on: - 'Cargo.lock' - 'bundler.toml' - 'tools/**' + # Mirrors the push filter. Without it a PR touching only the README would run a + # full build - the branch filter alone constrains where, not what. pull_request: branches: [ master ] + paths: + - 'src/**' + - 'xtask/**' + - 'Cargo.toml' + - 'Cargo.lock' + - 'bundler.toml' + - 'tools/**' workflow_dispatch: # Target the self-hosted Rust/audio runner registered to the `musiikki` org.