symfony-console-completion icon indicating copy to clipboard operation
symfony-console-completion copied to clipboard

Support for fish shell

Open Radiergummi opened this issue 5 years ago • 2 comments

As it seems, you don't currently support auto-completion for the fish shell. Fish is a well-established shell by now, I think (are there any statistics on shell usage?), so it'd make sense to add support.

I've just stumbled across this problem, but I really like your approach and would like to contribute a PR if there is any interest in supporting fish. As far as I've seen, both bash and zsh are implemented as string variables in the HookFactory currently.
Would it make sense to outsource this into a more generic Hooks\${ shellName }Factory approach?

Radiergummi avatar Nov 24 '19 10:11 Radiergummi

Happy to have Fish support added :+1:

CompletionCommand:: escapeForShell is probably in the wrong place responsibility-wise, so splitting out each supported shell into its own class sounds like a good direction.

stecman avatar Nov 24 '19 17:11 stecman

awesome :+1:
I'll see what I can come up with.

Radiergummi avatar Nov 25 '19 08:11 Radiergummi