Skip to content

chore: land prefetch.sh .claude exclusion on this branch #9502

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
koenvanderveen merged 1 commit into from
Sep 2, 2026
+4 −1
Merged

chore: land prefetch.sh .claude exclusion on this branch #9502

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion .claude/skills/syft-pr-review/prefetch.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,10 @@ gh pr view "$PR" --json files \
| sort -rn >"$OUT/files.txt"

# Lock files, notebooks and generated data are counted and skipped, not read.
grep -E '(\.lock|lock\.json|\.ipynb|\.min\.(js|css)|\.svg|\.png|\.csv|\.parquet)$' \
# `.claude/` too: agent tooling is not the PR author's work, and it rides along
# in the diff whenever a skill edit is landed on the branch before it lands on
# the base. Reviewing it would attribute it to the wrong person.
grep -E '(\.lock|lock\.json|\.ipynb|\.min\.(js|css)|\.svg|\.png|\.csv|\.parquet)$|(^| )\.claude/' \
"$OUT/files.txt" >"$OUT/files-skippable.txt" || true

# ----------------------------------------------------------------- history ---
Expand Down
Loading