Skip to content

Make globs case-insensitive like the search to close #716 - #773

Merged
mathiasrw merged 1 commit into
mainfrom
fix/716
Oct 23, 2026
Merged

mathiasrw merged 1 commit into
mainfrom
fix/716

Conversation

@mathiasrw

Copy link
Copy Markdown
Owner

Closes #716

Stacked on #772.

Change

Performance

**/*.md in a tree with 10,000 files (5,000 .md), 20 runs each with hyperfine:

case-sensitive case-insensitive
Node 72.7 ± 5.5 ms 68.3 ± 2.5 ms
Bun 42.3 ± 2.1 ms 43.1 ± 4.6 ms

There is no difference that can be measured.

Good to know

  • This is a breaking change, and it goes into v8.

Tests

  • test/js/fix/716.test.js: the input globs, --exclude-glob, and --void-ignore-case for both.
  • test/cli/run.sh: the same through the CLI.

@mathiasrw
mathiasrw added this pull request to stack #777 October 23, 2026 01:54
Base automatically changed from fix/710 to main October 23, 2026 02:00
@mathiasrw
mathiasrw merged commit 242f574 into main Oct 23, 2026
6 of 7 checks passed
@mathiasrw
mathiasrw deleted the fix/716 branch October 23, 2026 02:15
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.

Let globs be case-insensitive like the search

1 participant