Skip to content

Commit 82bec75

Browse files
cpcloudclaude
andcommitted
fix: remove explicit dorny base, rely on v3 default
dorny/paths-filter v3 already diffs against the repo default branch for non-default-branch pushes. Explicit base: main was redundant and would produce wrong baselines for backport PRs targeting release branches (all files seen as changed vs main). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 61f8d6a commit 82bec75

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

.github/workflows/ci.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -111,11 +111,10 @@ jobs:
111111
if: ${{ startsWith(github.ref_name, 'pull-request/') }}
112112
uses: dorny/paths-filter@d1c1ffe0248fe513906c8e24db8ea791d46f8590 # v3.0.3
113113
with:
114-
# Explicitly compare against the repository default branch. On push
115-
# events dorny/paths-filter defaults to diffing against the prior
116-
# commit, which is wrong for backport PRs targeting non-default
117-
# branches. Pinning base to main ensures consistent detection.
118-
base: main
114+
# dorny/paths-filter v3 on push to a non-default branch defaults to
115+
# diffing against the repository default branch. No explicit base is
116+
# needed; omitting it lets backport PRs targeting release branches
117+
# diff against their actual target rather than always against main.
119118
filters: |
120119
bindings:
121120
- 'cuda_bindings/**'

0 commit comments

Comments
 (0)