Suggest fish users to install sdkman-for-fish

[reitzig]

Posting this as a discussion starter.

Goal: In order to make SDKMAN! accessible to fish, install sdkman-for-fish after the installation was otherwise successful (as an option).

• Does not pester users without fish.
• Does not pester users which already have an sdk.fish file. (Would conflict.)
• Asks before installing.

Left to do: finish bash trickery; finalize wording.

Note:

1. The installation method is opinionated.
2. reitzig/sdkman-for-fish may have to mature a bit still.

[Markcial]

@reitzig I understand that you did that PR with the best of your intentions. But it won't simply work. That post-hook is intended to be run on bash (check on the shebang at the beginning of the line). And on bash, any of the commands or functions created for fish, won't be available.

[reitzig]

Ugh, quite right. Juggling multiple shells for a day apparently got to me. :D Did I miss anything else? (Besides the unimplemented parts, those are intentional for now.)

[Markcial]

Good thinking. was busy at the moment and i failed to advice you to do what you have already solved. Sorry for that.

Also i left a couple of issues on your repository for sdkman

[reitzig]

sdkman-for-fish has matured and I feel confident suggesting it to SDKMAN! users.

I updated the PR to propose a functional, skippable installation routine.

Concern: This makes the installer interactive. While the overlap between "uses fish" and "installs SDKMAN! in a non-interactive shell" is probably small, do we want to exclude it?
An alternative would be to just print the commands the user should execute.

Thoughts?