use fast-glob

[gurgunday]

Ref: https://github.com/isaacs/node-glob#comparison-to-other-javascript-glob-implementations

If you are reasonably sure that the patterns you will encounter are relatively simple, and want the absolutely fastest glob matcher out there, use fast-glob.

Checklist

• run npm run test and npm run benchmark
• tests and/or benchmarks are included
• documentation is changed or added
• commit message and code follows the Developer's Certification of Origin
  and the Code of conduct

[jsumners]

Given that this a module for use by a wide audience, how do you guarantee the caveat you mentioned about simple glob cases? Additionally, this is a cost paid at start up. Does it really need to be "fast" at the cost of accuracy?

[gurgunday]

I see your point, but in this case the glob pattern is simple

They are also both very popular

For the start-up time, we could help things with some refactoring, but at the end of the day, I just thought the API of the package was more suitable as it doesn't need to be promisifed

I don't have a strong opinion, we can leave this PR and I'll try to do some more testing and refactoring 🙏

[Eomm]

Does it really need to be "fast" at the cost of accuracy?

Yeah, I don't think it worth. Backend's first rule:

• it must work and it must be solid

Backend's 2nd rule:

• it must be fast

[gurgunday]

These are valid concerns. Closed with comment!

[Uzlopak]

@gurgunday

Could you have a look why glob is not upgrading?