Use Minimatch class for improved perf

[UltCombo]

While node-glob doesn't implement the ignore option, it would be faster to handle handle globs with the Minimatch class (see sindresorhus/globby#6 (comment) and the comment below).

[UltCombo]

Before I send a PR to fix this, does glob-stream's globs argument support anything other than strings? There's this typeof pattern !== 'string' and pattern instanceof RegExp, but there are no docs nor tests regarding these apparently. Can these be removed safely?

[yocontra]

@UltCombo We do support custom RegExps at the post-scan level https://github.com/wearefractal/glob-stream/blob/master/index.js#L104 I'd like to keep that if possible

[UltCombo]

@contra Oh I see. But https://github.com/wearefractal/glob-stream/blob/master/index.js#L109 shouldn't be return false;? Or are RegExps considered as negative globs for some reason? I can't seem to find any unit test using RegExps.

[yocontra]

@UltCombo All RegExps are treated as negative since they are used to filter post-scan which means they cannot be positive. You can't give a regex to node-glob so there is no way it could be positive. There probably aren't tests if you can't find any, but there should be

[UltCombo]

Oh I see, thanks for the explanation :)
I'll write some tests for those as well after I get this working.