reitzig / sdkman-for-fish

Adds support for SDKMAN! to fish
MIT License
280 stars 13 forks source link

Delete forcefully temp file for bashDump #42

Closed liuml07 closed 1 year ago

liuml07 commented 1 year ago

If the user sets alias alias rm "rm -i" to be protective about delete files, it will prompt every time we run sdk command. It's better to just forcefully delete the temp file.

Screenshot 2023-06-20 at 3 39 20 PM
reitzig commented 1 year ago

It's not feasible to account for all ways users may think to override any of the "basic" commands in use; see also #23. I recommend to not do that.

FWIW, the fix put forth in #37 -- using command -- seems more appropriate.