- πΊπΈ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
5 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
3 months ago 7:41am 26 August 2024 - First commit to issue fork.
-
graber β
committed b2511c19 on 8.x-2.x authored by
anybody β
Issue #3258682: Offer least destructive options first
-
graber β
committed b2511c19 on 8.x-2.x authored by
anybody β
- Status changed to Fixed
3 months ago 9:19am 26 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.