Added openssl completions#6845
Closed
exploide wants to merge 1 commit intofish-shell:masterfrom
exploide:openssl
Closed
Added openssl completions#6845exploide wants to merge 1 commit intofish-shell:masterfrom exploide:openssl
exploide wants to merge 1 commit intofish-shell:masterfrom
exploide:openssl
Conversation
Contributor
Author
|
Just force pushed an even shorter version avoiding one of the helper functions. |
Contributor
|
Neat, merged as 67eaefe |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
More than one time, I wished fish had autocompletions for
openssl. Butopensslwith all its dozens of subcommands and hundreds of options is so complex, I thought it would require an insane amount of work.Now, actually that's the shortest completion file I've ever written. 😆 It turns out openssl is pretty good in completing itself with subcommands and options.
I think it's far from being perfect since there is no completion for further arguments and there also is no description text, but it is still an improvement to the current non-completion.
Maybe it is a bit hacky. If you know whether this could be done in a better way, please let me know.
TODOs: