akarzim / zsh-docker-aliases

ZSH Docker aliases and functions
MIT License
204 stars 36 forks source link

Remove use of multiple single options #10

Closed MicahElliott closed 4 years ago

MicahElliott commented 4 years ago

Zsh's completion doesn't work well for some aliases that have multiple options grouped together, like for this case:

docker run -it -<TAB> # OOPS! gets a bunch of file names instead of options

But it's fine for:

docker run -i -t -<TAB>

So the aliases should rather use the second form. I'm new to these aliases, but I've just observed this with dkR -<TAB> and see that the double-single-option usage is common in these aliases, particularly the -it combo. I'll submit a PR if it you agree there's no harm in separating them.