Masafumi Koba
Masafumi Koba
To make the first step easier, we could start with the minimum feature: only supporting built-in formatters in a configuration. After it is released, this feature can be extended. ```js...
Okay. Let's implement the support for custom formatters in this PR. 👍🏼
> The "configuration not found" error message is now returned but the test is expecting an empty result. If I understand correctly, the new result seems better so we may...
@emmacharp Okay, let's sync this feature branch with the `main` branch and make this PR ready for review!
No need to test the uncovered line because of an edge case. 👍🏼
@romainmenke Thank you for opening the discussion. I totally agree with your thoughts. We should fix this problem in the next major version through deprecation minor releases. 👍🏼 Also, I...
Thanks for providing the type. That sounds good to me 👍🏼 I have the same concern about `line`. If we can give it up, I'd like to deprecate `line` and...
It makes sense to require only a node argument and make others optional 👍 Now I think this is ready to implement if you have no concerns.😀
Omitting `endIndex` seems pretty ambiguous to me, too. I think specifying both the start and end is much more obvious.
I'm in favor of @romainmenke's suggestion to add the `ignoreFunctions` secondary option. 👍🏼