Skip to content

Fix Windows exclude_patterns support + add non-regression test - #175

Merged
keith merged 2 commits into
keith:mainfrom
ArchangelX360:fix-windows-exclude-patterns
Oct 30, 2026
Merged

keith merged 2 commits into
keith:mainfrom
ArchangelX360:fix-windows-exclude-patterns

Conversation

@ArchangelX360

Copy link
Copy Markdown
Contributor

No description provided.

@ArchangelX360
ArchangelX360 force-pushed the fix-windows-exclude-patterns branch from cd199c8 to 6ca9702 Compare October 27, 2026 13:56
@ArchangelX360
ArchangelX360 force-pushed the fix-windows-exclude-patterns branch from 6ca9702 to f0f03ce Compare October 27, 2026 13:57
Walk the workspace without entering excluded directories, so patterns
like `./node_modules` prune the walk as with find instead of filtering
files after a full recursive scan.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ArchangelX360
ArchangelX360 force-pushed the fix-windows-exclude-patterns branch 2 times, most recently from 025b660 to 7b0c1d0 Compare October 28, 2026 14:14
@ArchangelX360

Copy link
Copy Markdown
Contributor Author

@keith Maybe it's better to invest into making that bazel-contrib/buildtools#1486 merged actually instead of reviewing and merging mine.

@keith
keith merged commit dc2c900 into keith:main Oct 30, 2026
77 of 80 checks passed
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.

2 participants