-
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge support #1
Comments
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. |
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 would be not ignored in that scenario with this library. Unless the library is buggy we try to parse ignore files like git would. |
OK, I tried it, it works good:
|
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 🤔 |
Could you please tell me is merge supported? Similar to this one:
It was merged to:
The text was updated successfully, but these errors were encountered: