Skip to content

fix: report path-separator patterns consistently on Unix#1929

Open
cl2t wants to merge 1 commit intosharkdp:masterfrom
cl2t:fix-path-separator-pattern-error
Open

fix: report path-separator patterns consistently on Unix#1929
cl2t wants to merge 1 commit intosharkdp:masterfrom
cl2t:fix-path-separator-pattern-error

Conversation

@cl2t
Copy link
Copy Markdown

@cl2t cl2t commented Mar 18, 2026

Summary

Fixes #1873.

Report the path-separator pattern error consistently on Unix even when the pattern is not an existing directory.
Windows behavior is left unchanged because \ can also be a regex escape.

Testing

  • cargo test test_path_pattern_with_separator_reports_error_for_nonexistent_directory -- --exact
  • cargo test --test tests
  • cargo fmt --all --check

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.

[BUG] The 'pattern contains path separator' error is only shown when pattern is an existing directory

1 participant