Skip to content
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

Merged
merged 12 commits into from
Sep 1, 2024

Conversation

RobLoach
Copy link
Contributor

@RobLoach RobLoach commented Aug 26, 2024

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.

asdf plugin add nodejs
asdf install nodejs latest

@RobLoach RobLoach changed the title Add asdf plugin plugins/asdf: Add asdf plugin Aug 26, 2024
@RobLoach RobLoach changed the title plugins/asdf: Add asdf plugin plugins/asdf: Add asdf extendable version manager plugin Aug 26, 2024
Copy link
Contributor

@akinomyoga akinomyoga left a 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.

completions/asdf.completion.sh Outdated Show resolved Hide resolved
plugins/asdf/asdf.plugin.sh Outdated Show resolved Hide resolved
plugins/asdf/asdf.plugin.sh Outdated Show resolved Hide resolved
plugins/asdf/asdf.plugin.sh Outdated Show resolved Hide resolved
plugins/asdf/README.md Outdated Show resolved Hide resolved
RobLoach and others added 2 commits August 26, 2024 21:27
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
@RobLoach
Copy link
Contributor Author

Thanks a lot for the review, I'll fix these up.

RobLoach and others added 2 commits August 26, 2024 21:35
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
plugins/asdf/README.md Outdated Show resolved Hide resolved
@RobLoach
Copy link
Contributor Author

I didn't consider the commit message format here, so when merging, feel free to squash on merge. Thanks a lot for the reviews.

Copy link
Contributor

@akinomyoga akinomyoga left a 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

plugins/asdf/asdf.plugin.sh Outdated Show resolved Hide resolved
@akinomyoga akinomyoga merged commit a3720d8 into ohmybash:master Sep 1, 2024
4 checks passed
@akinomyoga
Copy link
Contributor

Thank you.

@RobLoach
Copy link
Contributor Author

RobLoach commented Sep 1, 2024

Thank you so much for the reviews and merge 👍 ... Happy to have this in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants