Expose the rest of `H.builtin_textobjects` so users can assign them to a different key about mini.nvim HOT 4 CLOSED

Thanks for the suggestion!

The "argument" and "function_call" textobjects are exposed in gen_spec because they might be configured to use some different hyper parameters.

Currently I don't think exposing the whole 'builtin_textobjects' is a good idea from maintenance perspective while bringing not huge amount of dividends (setting them up explicitly still works). But I'll think about it.

from mini.nvim.

Sorry, I don't think it is too much for users to have explicit definitions for textobjects that they want to use. And as already was said, "argument" and "function_call" may need configuration so having them in gen_spec makes sense.

Closing as not planned.

from mini.nvim.

That's disappointing to hear, maybe i lack perspective but I didn't think that there would be such a maintenance burden from exposing whats already implemented. I just want to use the existing builtin textobjects with a different key; now I have to repeat the esoteric definitions and lose out on updates unless I manually check whether they have changed.

from mini.nvim.

... but I didn't think that there would be such a maintenance burden from exposing whats already implemented.

It is implemented as an internal helper right now which means that it is free to be transformed in any way that might be necessary. Exporting it makes it harder to do.

Plus documenting and testing all of those 15 extra specifications takes space and effort.

Plus if this is done in 'mini.ai', then it is reasonable to be done in 'mini.surround', which adds 13 more surrounding specifications to export, document, and test.

As both 'mini.ai' and 'mini.surround' are already quite big and benefits are not huge, I'd better not export them.

from mini.nvim.