Incorrect matching with curly braces and globstar

[mrmlnc]

Source

• Difference behavior for dot files on windows and linux mrmlnc/fast-glob#159

Actual behavior

The following filepathes is not matched by minimatch, micromatch and picomatch for the {file.txt,directory/**/*} pattern.

directory/.test.txt
directory/test.txt

But if you remove the /* part of the pattern, everything works correctly.

Expected behavior

Will be matched or documented :)

Code sample

Works fine:

picomatch.makeRe('{file.txt,directory/**}', { dot: true });

• https://runkit.com/mrmlnc/5c6db74af4685d0012b8a037

Works bad:

picomatch.makeRe('{file.txt,directory/**/*}', { dot: true });

• https://runkit.com/mrmlnc/5c6db7a2629c26001267cf8d

[jonschlinkert]

Thanks for the issue. Good timing too, since I'm working on this again this week, to publish the next version of micromatch.

Edit: actually I'm wondering if that is the correct result. I'm pretty sure that when you specify **/*, that means "any number of directories, followed by at least one more directory". Also, I'm pretty sure that /* usually means "one directory", not "zero or one". But given that there is no actual specification for globs and it works differently in different tools, I'm open to feedback.

Edit 2: nevermind, ignore my previous comment, I was thinking the glob would have a file after **/*, but it wouldn't. You were correct.