-
-
Notifications
You must be signed in to change notification settings - Fork 665
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
plugins/asdf: Add asdf
extendable version manager plugin
#593
Conversation
asdf
pluginasdf
extendable version manager plugin
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.
It's roughly based off of the original pull request, moves to the bash syntax, and allows checking for the
ASDF_DIR
variable before hitting the file system.
Thank you. Could you mention the original author (https://github.com/prodrigues1912) and URL of PR #310 in the code comment around the top of asdf.plugin.sh
? Since it will not remain in the git history, we need to leave the information manually.
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
Thanks a lot for the review, I'll fix these up. |
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
I didn't consider the commit message format here, so when merging, feel free to squash on merge. Thanks a lot for the reviews. |
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.
Thank you for updating
Thank you. |
Thank you so much for the reviews and merge 👍 ... Happy to have this in. |
This adds a plugin to load the asdf version manager through oh-my-bash. It's roughly based off of the original pull request, moves to the bash syntax, and allows checking for the
ASDF_DIR
variable before hitting the file system.