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

Clarify the effects of path separator #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Clarify the effects of path separator #49

wants to merge 1 commit into from

Conversation

xuhdev
Copy link
Member

@xuhdev xuhdev commented Jun 18, 2024

This is consistent with the rules from .gitignore: https://git-scm.com/docs/gitignore

If there is a separator at the beginning or middle (or both) of the
pattern, then the pattern is relative to the directory level of the
particular .gitignore file itself. Otherwise the pattern may also match
at any level below the .gitignore level.

Since we intend to be consistent with .gitignore, this part of the spec has been unclear.

Fix editorconfig/editorconfig#509

This is consistent with the rules from `.gitignore`: <https://git-scm.com/docs/gitignore>

> If there is a separator at the beginning or middle (or both) of the
> pattern, then the pattern is relative to the directory level of the
> particular .gitignore file itself. Otherwise the pattern may also match
> at any level below the .gitignore level.

Since we intend to be consistent with `.gitignore`, this part of the
spec has been unclear.

Fix editorconfig/editorconfig#509
@xuhdev xuhdev requested a review from cxw42 June 18, 2024 21:30
@xuhdev
Copy link
Member Author

xuhdev commented Jun 18, 2024

And we have been unclear with slashes at the end because it didn't matter as matching directories is pointless...

@xuhdev xuhdev requested a review from florianb June 20, 2024 21:46
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.

Meaning of a glob like a/*.c
1 participant