-
-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Labels
changes/sourceSource file changesSource file changeskind/featureNew feature or requestNew feature or requestpriority/4Lower priorityLower prioritystatus/readyReady to workReady to work
Description
Contact Details
Feature Description
Improve the command-installed function by supporting more types of command installations, particularly aliases and package bundles/libraries.
Why?
There are managed packages (omz, nvm, moreutils) that command-installed currently, incorrectly, returns false.
What should happen?
This is not a bug because we handle it in different ways right now, but they are bespoke workarounds.
command-installed needs to support different package installations, this is probably most easily done by importing aliases then testing for them, using the package managers themselves, using command -v, and using which and ||ing the result.
Covers this TODO, and this TODO.
Steps to Test
- Install omz, nvm, and moreutils.
general-command-installed -c omzshould returntruegeneral-command-installed -c nvmshould returntruegeneral-command-installed -c moreutilsshould returntrue
Code of Conduct
- I agree to follow this project's Code of Conduct
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
changes/sourceSource file changesSource file changeskind/featureNew feature or requestNew feature or requestpriority/4Lower priorityLower prioritystatus/readyReady to workReady to work