Skip to content
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

Options merge #107

Open
coderaiser opened this issue Nov 22, 2023 · 4 comments
Open

Options merge #107

coderaiser opened this issue Nov 22, 2023 · 4 comments

Comments

@coderaiser
Copy link

Would be great to have ability to merge options, so if I have:

tmp
**/coverage
!**/coverage

It was merged to:

tmp

Is it possible?

@kaelzhang
Copy link
Owner

kaelzhang commented Nov 25, 2023

I can't understand your question. could you provide minimum slices of code and the expected result?

@coderaiser
Copy link
Author

Sure coderaiser/putout@4a83bb8

@kaelzhang
Copy link
Owner

Is it for performance purposes?

But why do we write such a silly ignore rule set as ['**/coverage', '!**/coverage']?

@coderaiser
Copy link
Author

Good question! There is default ignore rules in putout.json which says to ignore coverage directory. And user can create it's own .putout.json file, and would be great if he can override it changes.

I need it because I have a big monorepo, with a lot of coverage directories in each module, but I don't really need it, so I write a rule do remove them, so I need to override default change.

Also someone can have it's own preferences about things that should be ignored, and ability to override defaults is always good to have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants