-
Notifications
You must be signed in to change notification settings - Fork 436
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Infer shells automatically, and use
versions based on the current working directory (optional)
#68
Conversation
2a596c4
to
e10c50c
Compare
|
||
alias cd=__fnmcd | ||
|} | ||
| Fish => {| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @elliottsj! do you want to take a look? I know it works - but do you think this is a good way to do it on fish?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! It's a little beyond my expertise, but from a cursory Google search, --on-variable PWD
seems like a great way to do this.
25fd17c
to
628e2b8
Compare
|
||
alias cd=__fnmcd | ||
|} | ||
| Fish => {| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! It's a little beyond my expertise, but from a cursory Google search, --on-variable PWD
seems like a great way to do this.
use
versions based on the current working directory (optional)use
versions based on the current working directory (optional)
* Add the ability to `use` on every pwd change (bash, zsh) * infers shell type automatically. Fixes #20
Add the ability to
use
on every pwd change (bash, zsh). Also, infers shell type automatically. Fixes #20Thanks @yardnsm for the
zsh
snippet!Note: in order to use it, everyone who uses fnm needs to add double-quotes around the "env":
for Fish we moved to use
source
instead ofeval
: