Navigation Menu

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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added openssl completions #6845

Closed
wants to merge 1 commit into from
Closed

Added openssl completions #6845

wants to merge 1 commit into from

Conversation

exploide
Copy link
Contributor

Description

More than one time, I wished fish had autocompletions for openssl. But openssl with 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:

  • Changes to fish usage are reflected in user documentation/manpages.
  • Tests have been added for regressions fixed
  • User-visible changes noted in CHANGELOG.md

@exploide
Copy link
Contributor Author

exploide commented Apr 1, 2020

Just force pushed an even shorter version avoiding one of the helper functions.

@krobelus
Copy link
Member

krobelus commented Apr 2, 2020

Neat, merged as 67eaefe

@krobelus krobelus closed this Apr 2, 2020
@krobelus krobelus added this to the fish 3.2.0 milestone Apr 2, 2020
@exploide exploide deleted the openssl branch April 12, 2020 14:05
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants