Merge support about fast-ignore HOT 5 CLOSED

If I'm understanding what you are asking for: ignore files are not optimized to remove kinda self-cancelling rules like that, but if you repeat those 2 lines 1 million times they won't slow down matching because they end up populating the same nodes internally, like they kinda overwrite each other, so I guess there is some kind of implicit merging.

from fast-ignore.

I’m talking not about slowdown, but about overriding ignore. It useful when something was ignored by default and you want to stop ignoring it. How it will work right now for such case:

**/coverage
!**/coverage

coverage is ignored or not when it used by your library?

from fast-ignore.

Coverage would be not ignored in that scenario with this library. Unless the library is buggy we try to parse ignore files like git would.

from fast-ignore.

OK, I tried it, it works good:

**/coverage
!**/coverage

coverage ignored and:

!**/coverage
**/coverage

coverage not ignored. Is it possible to produce CommonJS? I need to use this library inside ESLint plugin which is sync CommonJS only.

from fast-ignore.

Sorry I've moved fully to ESM to avoid all the complications with supporting both at the same time. You can bundle it to CJS inside your package or something if you really need to import it synchronously 🤔

from fast-ignore.