-
Notifications
You must be signed in to change notification settings - Fork 903
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.
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
exclude pattern with *
in file name incorrectly matches entire file path
#6262
Comments
We use @zanieb - could use a second opinion, any objection to changing that? |
What are you expected to do to get the desired behavior without I'm a little confused that this is the default in It'd be a breaking change for us to change this, but I think it'd probably be best to match Unix shell behavior. |
On mobile so apologies for short reply. Globbing is pretty tricky because it depends on the context in which it is run. For example, at a shell, you might expect So whether |
Thanks! I'm impressed that you can manage code formatting on mobile. |
We have kind of a weird thing going on, which I think I followed from Flake8 or Black. When given a pattern like So it may make sense to turn on |
👍 |
I'm not sure about Android, but if you long-press the |
On Android, the long press variations for |
Came here because I was having an issue with the globs unexpectedly including subdirectories. In my case, I am trying to exclude
I added: [tool.ruff]
extend-exclude = ["tests/*.py"] I was surprised to find out that this excluded all Python files in |
this incorrectly excludes the file
foo/bar/baz.py
.since
excludes
takes globs (as mentioned here), i would expect it to only match python files with names starting withfoo
.i'm on windows (which uses
\
as path separators instead of/
), that may be relatedThe text was updated successfully, but these errors were encountered: