- 🇺🇸United States amanire
Yes! I agree this should at least be configurable. In the meantime, I'll use the patch.
- 🇩🇪Germany Anybody Porta Westfalica
+1 on the suggestion. Any plan to finish this @joshmiller?
- Status changed to Needs review
6 months ago 1:28pm 4 July 2024 - 🇺🇸United States joshmiller Indianapolis, Indiana, USA
Sure! Perhaps I can update the tests, and make this a configurable option. I'll update in this thread if and when I start working. Until then, it's open for anyone else to do those things with the open merge request.
- 🇩🇪Germany Anybody Porta Westfalica
@joshmiller a configurable action would be far to much. If someone really needs to alter this (which I think will never happen - YAGNI & KISS), a hook can be used. Just solve it usefully by default.
Thanks! :) - 🇺🇸United States amanire
Just to clarify my comment in #6. I meant that it should at least be configurable if the default was not least destructive. But if the default is changed, I don't care if it's not configurable. Thank you for moving this forward!
- Status changed to RTBC
4 months ago 7:41am 26 August 2024 - First commit to issue fork.
- Status changed to Fixed
4 months ago 9:19am 26 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.