Skip to content
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

Globset is confusing for top-level folder inclusion #107

Open
magnalite opened this issue Feb 19, 2023 · 0 comments
Open

Globset is confusing for top-level folder inclusion #107

magnalite opened this issue Feb 19, 2023 · 0 comments
Labels
essential feature A feature essential to wally supporting all typical use cases. All must be closed before 1.0.0.

Comments

@magnalite
Copy link
Member

To include a top-level folder you need to do

include = ["folder", "folder/**"]
exclude = ["**"]

While this is correct behaviour according to globset For example, foo/** matches foo/a and foo/a/b, but not foo. this is confusing as an end user. There must be some way to make this more intuitive.

@magnalite magnalite added the essential feature A feature essential to wally supporting all typical use cases. All must be closed before 1.0.0. label Jun 13, 2023
@magnalite magnalite added this to the Essential feature complete milestone Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
essential feature A feature essential to wally supporting all typical use cases. All must be closed before 1.0.0.
Projects
None yet
Development

No branches or pull requests

1 participant