Skip to content

Conversation

xt0rted
Copy link
Owner

@xt0rted xt0rted commented Dec 4, 2022

This adjusts how globbing works with script names so : is treated as /. To do this we swap those two characters before comparing them which is also how npm-run-all does it.

Changing the behavior of : allows the following scenarios now:

  1. foo:* will match foo:bar but not foo:bar:baz
  2. foo:*:baz will match foo:bar:baz
  3. foo:** will match foo:bar and foo:bar:baz

@xt0rted xt0rted added the enhancement New feature or request label Dec 4, 2022
@xt0rted xt0rted force-pushed the globbing-separator branch from d24a286 to 0a39ec9 Compare December 4, 2022 21:49
@xt0rted xt0rted marked this pull request as ready for review December 6, 2022 04:58
@xt0rted xt0rted force-pushed the globbing-separator branch from 0a39ec9 to d5a10ae Compare December 6, 2022 06:25
@xt0rted xt0rted merged commit 3aa9ab1 into main Dec 6, 2022
@xt0rted xt0rted deleted the globbing-separator branch December 6, 2022 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant