fnm icon indicating copy to clipboard operation
fnm copied to clipboard

Deprecate shell inference

Open Schniz opened this issue 4 years ago • 3 comments

Something to consider. It's out of fnm's scope anyway. We know on what shell we're installing it into, and projects like Starship don't infer shells too.

Schniz avatar Jan 15 '21 08:01 Schniz

If all commands would accept the --shell=fish argument, a user could just create this alias:

alias f="fnm --shell=fish"

(Or something like it, in their shell). Currently however, it doesn't seem like all commands accept this argument, so the alias doesn't work. It fails with fnm use for example. Or am I misunderstanding how this works?

ismay avatar Apr 08 '21 13:04 ismay

Yes, but only env and completions, which are commands that should only appear in the users' shell profile, need the shell argument. So I don't see why it needs to be a global config argument 😄

Schniz avatar Apr 08 '21 13:04 Schniz

Ah I see. So I misunderstood how to use it. Ok, thanks for the explanation! In that case deprecating shell inference sounds good to me 👍

ismay avatar Apr 08 '21 13:04 ismay