-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
tests: text and path glob path matching #33104
tests: text and path glob path matching #33104
Conversation
Hi @jaellio, welcome and thank you for your contribution. We will try to review your Pull Request as quickly as possible. In the meantime, please take a look at the contribution guidelines if you have not done so already. |
Please merge the main branch to fix CI breakage. /wait |
97f8cc7
to
8a016a2
Compare
@yanavlasov Updated! |
Please merge past 7fdde72 Seems like you did not pick up all the necessary updates. /wait |
path pattern matching. Signed-off-by: Jackie Elliott <jaellio@microsoft.com>
Signed-off-by: Jackie Elliott <jaellio@microsoft.com>
Signed-off-by: Jackie Elliott <jaellio@microsoft.com>
bd9dd57
to
6333971
Compare
Gotcha, my bad! Updated! |
Adds additional test cases for path and text glob path pattern matching. --------- Signed-off-by: Jackie Elliott <jaellio@microsoft.com>
Commit Message: Adds additional test cases for path and text glob path pattern matching.
Additional Description: Specifically verifies that
=**
or**
must be the last operator in the path pattern match.Risk Level: Low
Testing: Ran new tests locally.
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: